Skip to content

add stateless http transport support#213

Open
Edison-A-N wants to merge 1 commit intotadata-org:mainfrom
Edison-A-N:feature/stateless-flag
Open

add stateless http transport support#213
Edison-A-N wants to merge 1 commit intotadata-org:mainfrom
Edison-A-N:feature/stateless-flag

Conversation

@Edison-A-N
Copy link
Copy Markdown

Describe your changes

add stateless http transport support

Issue ticket number and link (if applicable)

#209

Screenshots of the feature / bugfix

Checklist before requesting a review

  • Added relevant tests
  • Run ruff & mypy
  • All tests pass

@Edison-A-N Edison-A-N force-pushed the feature/stateless-flag branch from 7abea08 to b891c21 Compare August 15, 2025 08:10
@buhaiqing
Copy link
Copy Markdown

I hope this feature can be officially released as soon as possible

@frouzbeh
Copy link
Copy Markdown

Hi, Is this pr complete? I am trying to use langchain mcp-adapter client, and it seems it only supports stateless streamable_http.

@Edison-A-N
Copy link
Copy Markdown
Author

Hi @frouzbeh

You can test the changes in either branch—both are already rebased on the latest main:

Both branches contain the feature proposed in this PR.

@jakegt1
Copy link
Copy Markdown

jakegt1 commented Aug 27, 2025

this really would be great to have merged as atm you simply cannot use this with gunicorn at all.

@Edison-A-N
Copy link
Copy Markdown
Author

You're right — as I mentioned in this comment, the current implementation breaks under Gunicorn with multi workers.

@brendanshanahan
Copy link
Copy Markdown

You're right — as I mentioned in this comment, the current implementation breaks under Gunicorn with multi workers.

it would be great to get this merged. i'm hitting the same wall with a multi-worker gunicorn application and switching it to single-worker is not an option

@jakegt1
Copy link
Copy Markdown

jakegt1 commented Sep 2, 2025

@itay-tadata sorry for the ping here, but this just seems a really useful feature for the project. Do you mind taking a look if you have the time?

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Derican
Copy link
Copy Markdown

Derican commented Oct 11, 2025

I noticed that this PR contains a feature that would be very helpful for my work. May I kindly ask if there is an estimated timeline for when it might be merged into the main release?

@jakegt1
Copy link
Copy Markdown

jakegt1 commented Oct 22, 2025

I noticed that this PR contains a feature that would be very helpful for my work. May I kindly ask if there is an estimated timeline for when it might be merged into the main release?

i would recommend just forking it yourself honestly. Currently this system is unworkable if you use gunicorn until stateless is enabled.

@Ashes47
Copy link
Copy Markdown

Ashes47 commented Oct 27, 2025

Can we please merge this pull request, i am facing a similar issue :)

@AbdulEmad
Copy link
Copy Markdown

Guys LGTM!!!!!!

@ltalirz
Copy link
Copy Markdown

ltalirz commented Mar 24, 2026

hm... just realizing the package has not seen a commit since 7 months; more than the time from first to last commit.

Is there a better way add mcp to a fastapi nowadays?

@Edison-A-N
Copy link
Copy Markdown
Author

@Italirz +1 on the maintenance concern.

I've been using goFastMCP instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants