-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Support initialize requests in middleware
#1546
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
Conversation
b780717 to
9c13ddf
Compare
We've skipped this test on windows, let's see if it helps now
|
Hey @jlowin this library has been amazing to use! I'm working on figuring out a solution for horizontally scaling while still being able to log usage across sessions and my current plan is:
This middleware looks perfect for step 3 (and I'd much rather stay within the FastMCP ecosystem), so I wanted to see if I could help with getting merged or if you were considering other approaches that might solve the same problem. |
|
Any help you need to get this merged? This is a feature I have been waiting for. |
This PR adapts part of #1215 to support initialize requests in FastMCP middleware. In order to do this, we have to introduce a new low-level
Sessionobject because initialize is handled outside the normalhandlerapproach that FastMCP generally relies on.