-
Notifications
You must be signed in to change notification settings - Fork 380
Feature: Accept Config Variables via Request Headers and Query Parameters (Multi-User) #239
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
This is a merge commit the virtual branches in your workspace. Due to GitButler managing multiple virtual branches, you cannot switch back and forth between git branches and virtual branches easily. If you switch to another branch, GitButler will need to be reinitialized. If you commit on this branch, GitButler will throw it away. Here are the branches that are currently applied: - Lane (refs/gitbutler/Lane) branch head: 62ee847 - src/mcp_atlassian/__init__.py For more information about what we're doing here, check out our docs: https://docs.gitbutler.com/features/virtual-branches/integration-branch
This is placeholder commit and will be replaced by a merge of yourvirtual branches. Due to GitButler managing multiple virtual branches, you cannot switch back and forth between git branches and virtual branches easily. If you switch to another branch, GitButler will need to be reinitialized. If you commit on this branch, GitButler will throw it away. For more information about what we're doing here, check out our docs: https://docs.gitbutler.com/features/virtual-branches/integration-branch
I get this
|
Hi @dsaad68, Thank you for the excellent work on this PR! The use of However, we've decided to hold off on merging multi-user features based on the current HTTP+SSE transport. The official MCP specification has adopted the new "Streamable HTTP" transport (#206) as the way forward. Our project strategy is to align with this standard for better long-term scalability and compatibility. Implementing multi-user support requires updates to the core Merging this SSE-based solution now would necessitate significant refactoring once the SDK is updated. To avoid this duplicate work, we plan to implement multi-user support directly on the Streamable HTTP standard once the SDK is ready. We truly value the architectural foundation laid here and hope to leverage these concepts in the future Streamable HTTP implementation. We'll be closing this PR for now due to the shift in the underlying MCP transport standard. Thanks again for your substantial contribution and understanding! |
I will close this PR. |
✨ Proposal: Extend MCP Server to Accept Config Variables via Request Headers
This PR introduces a work-in-progress proposal to extend the current MCP server by allowing configuration-related variables to be passed through request headers or query parameters in the request. This proposal addresses issue #205 and builds on #224. Some parts of this implementation are inspired by work from @maxxrdrgz.
✅ What’s included:
ContextVar
integration: Utilizes Python’sContextVar
to dynamically capture and manage configuration variables provided in the URL query string.🔧 Supported headers and query parameters:
The following headers are currently supported:
How to run the server for example:
How to add to the client (for example VS Code):
Query parameters from URL:
📝 To-Dos