-
Notifications
You must be signed in to change notification settings - Fork 851
add stateless http transport support #213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
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. |
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