Skip to content

Comments

Fix MCP multi user server#834

Merged
vnegrisolo merged 1 commit intomasterfrom
fix-mcp-multi-user-server
Oct 30, 2025
Merged

Fix MCP multi user server#834
vnegrisolo merged 1 commit intomasterfrom
fix-mcp-multi-user-server

Conversation

@vnegrisolo
Copy link
Member

@vnegrisolo vnegrisolo commented Oct 30, 2025

It looks like the authentication logic inside the server init doesn't handle multiple users.
Changing to do the auth logic inside each tool by using the frame's transport request headers fixes the issue.

See this issue zoedsoupe/anubis-mcp#29

Co-authored-by: Vinicius Negrisolo <vinicius.negrisolo@hashrocket.com>
Co-authored-by: Gabriel Reis <gabriel.reis@hashrocket.com>
@vnegrisolo vnegrisolo self-assigned this Oct 30, 2025
@greis greis self-requested a review October 30, 2025 20:18
@vnegrisolo vnegrisolo merged commit 7ee1650 into master Oct 30, 2025
3 checks passed
@vnegrisolo vnegrisolo deleted the fix-mcp-multi-user-server branch October 30, 2025 20:19
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.

3 participants