Skip to content

oauth: add tenant suffix to oauth flow#1096

Open
howardjohn wants to merge 1 commit intomodelcontextprotocol:mainfrom
howardjohn:oauth/correct-tenant
Open

oauth: add tenant suffix to oauth flow#1096
howardjohn wants to merge 1 commit intomodelcontextprotocol:mainfrom
howardjohn:oauth/correct-tenant

Conversation

@howardjohn
Copy link
Contributor

Summary

Fix #1095

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Refactoring (no functional changes)
  • Test updates
  • Build/CI improvements

Changes Made

This adds spec compliant Authorization Server Metadata Discovery

Related Issues

Fix #1095

Testing

  • Tested in UI mode
  • [] Tested in CLI mode
  • Tested with STDIO transport
  • Tested with SSE transport
  • Tested with Streamable HTTP transport
  • Added/updated automated tests
  • Manual testing performed

Test Results and/or Instructions

Screenshots are encouraged to share your testing results for this change.

Checklist

  • Code follows the style guidelines (ran npm run prettier-fix)
  • Self-review completed
  • Code is commented where necessary
  • Documentation updated (README, comments, etc.)

Breaking Changes

Additional Context

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.

Improper ​ Authorization Server Metadata Discovery

1 participant

Comments