Skip to content

Feature/authentication system #43

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

Merged
merged 19 commits into from
Jun 1, 2025
Merged

Conversation

Rodriguespn
Copy link
Collaborator

@Rodriguespn Rodriguespn commented May 30, 2025

Summary

This pull request adds a new OAuth flow to the MCP server for remote transports (sse and http). It introduces changes to how authentication and organization context are handled for remote and local environments.

Changes

  • OAuth support for remote transports
  • For remote (SSE and HTTP) transports, the server now supports OAuth.
  • The access token is extracted from the Authorization header in incoming HTTP requests.
  • API key required for local version
  • The local version now requires an API key to function, as OAuth is only available for remote transports.
  • The API key includes the organization_id, so there is no need to specify the org ID separately in local mode.
  • Environment-based organization ID for the remote version
  • The organization_id must be provided via an environment variable (ORGANIZATION_ID) for remote usage.

Breaking Changes

  • Local usage now requires an API key.
  • organization_id must be set via an environment variable for remote usage.
  • Clients must be updated to conform to the new authentication mechanisms depending on the transport type.

@Rodriguespn Rodriguespn self-assigned this May 30, 2025
@Rodriguespn Rodriguespn added the enhancement New feature or request label May 30, 2025
@Rodriguespn Rodriguespn merged commit aa0af3a into main Jun 1, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant