add stateless http transport support#213
Conversation
7abea08 to
b891c21
Compare
|
I hope this feature can be officially released as soon as possible |
|
Hi, Is this pr complete? I am trying to use langchain mcp-adapter client, and it seems it only supports stateless streamable_http. |
|
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. |
|
this really would be great to have merged as atm you simply cannot use this with gunicorn at all. |
|
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 |
|
@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 Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
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. |
|
Can we please merge this pull request, i am facing a similar issue :) |
|
Guys LGTM!!!!!! |
|
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? |
|
@Italirz +1 on the maintenance concern. I've been using goFastMCP instead |
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