-
Notifications
You must be signed in to change notification settings - Fork 432
[BUG] UVicorn generates error 404 session id not found when starting multiple workers #56
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
Comments
I'll be working on this issue |
thanks @tonderflash |
@surki I'm considering implementing a shared storage mechanism for SSE sessions that would work with multiple workers/pods to solve this issue. Do you have any specific preferences or restrictions regarding storage solutions (Redis, databases, etc.) that I should take into account for the use case? |
@tonderflash Perhaps we should have some kind of common interface for session management. Then we could have few implementations to start with, redis or so (we are looking at leveraging redis since it is already present in our infra). We should also think about purging stale entries, what happens if the target instance is down etc |
Any updates on this ? |
Uh oh!
There was an error while loading. Please reload this page.
Describe the bug
When using uvicorn to start multiple workers, the MCP server will report an error 404 session id Not Found due to session id issues
To Reproduce
Error log
System Info
Python 3.12.5
The text was updated successfully, but these errors were encountered: