Skip to content

Conversation

@wzxxing
Copy link
Contributor

@wzxxing wzxxing commented Nov 17, 2025

https://gofastmcp.com/servers/proxy#session-isolation-&-concurrency

Summary

Changes

Please provide a summary of what's being changed

The client passed to the proxy needs to be already connected, otherwise the proxy will create a new session to connection for every request

reference: https://gofastmcp.com/servers/proxy#session-isolation-&-concurrency

Fresh Sessions

When you pass a disconnected client (which is the normal case), each request gets its own isolated backend session:

User experience

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have reviewed the contributing guidelines
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Is this a breaking change? (Y/N)

  • Yes
  • No

Please add details about how this change was tested.

  • Did integration tests succeed?
  • If the feature is a new use case, is it necessary to add a new integration test case?

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@wzxxing wzxxing force-pushed the wzxxing/fix-session branch from 5b78cda to 379ede3 Compare November 17, 2025 14:05
@wzxxing wzxxing marked this pull request as ready for review November 17, 2025 14:08
@wzxxing wzxxing requested a review from a team as a code owner November 17, 2025 14:08
@wzxxing wzxxing requested review from harv-aws and vtols November 17, 2025 14:08
@wzxxing wzxxing enabled auto-merge (squash) November 17, 2025 14:20
@wzxxing wzxxing merged commit da18ca7 into main Nov 17, 2025
7 checks passed
@wzxxing wzxxing deleted the wzxxing/fix-session branch November 17, 2025 14:21
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.

4 participants