Skip to content

mcp: take into account the request subject when resuming MCP sessions - #2740

Open
nacx wants to merge 3 commits into
mainfrom
mcp-session-resume
Open

nacx wants to merge 3 commits into
mainfrom
mcp-session-resume

Conversation

@nacx

@nacx nacx commented Sep 25, 2026

Copy link
Copy Markdown
Member

Description

Update the MCP implementation to always check the request subject when resuming MCP sessions.

Related Issues/PRs (if applicable)

N/A

Special notes for reviewers (if applicable)

N/A

Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
@nacx
nacx requested a review from a team as a code owner September 25, 2026 18:56
@netlify

netlify Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for theagentrouter canceled.

Name Link
🔨 Latest commit d158c5f
🔍 Latest deploy log https://app.netlify.com/projects/theagentrouter/deploys/6ab6cf41a62d650008006de6

@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/mcpproxy/legacy.go 88.88% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
@nacx

nacx commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

/retest

1 similar comment
@nacx

nacx commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

/retest

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.

1 participant