Skip to content

fix: persist auth method switch from OAuth to Bearer/none in MCP catalog#3083

Open
ozeranskii wants to merge 1 commit intoarchestra-ai:mainfrom
ozeranskii:fix/mcp-oauth-to-bearer-switch
Open

fix: persist auth method switch from OAuth to Bearer/none in MCP catalog#3083
ozeranskii wants to merge 1 commit intoarchestra-ai:mainfrom
ozeranskii:fix/mcp-oauth-to-bearer-switch

Conversation

@ozeranskii
Copy link
Contributor

Summary

  • Fix frontend transformFormToApiData using undefined instead of null when clearing oauthConfig - JSON.stringify() strips undefined, so the backend never receives the field and the old OAuth config stays in the database
  • Add backend cleanup of orphaned OAuth client secret (clientSecretId) when oauthConfig is explicitly set to null in the PUT handler
  • Add 5 frontend tests for transformFormToApiData covering all auth methods and JSON serialization
  • Add 2 backend tests for OAuth-to-Bearer and OAuth-to-none switching at the model level

Closes #3076

@London-Cat
Copy link
Collaborator

London-Cat commented Mar 2, 2026

📊 Reputation Summary

User Rep Pull Requests Activity Assigned Core Reactions
ozeranskii ⚡ 3 0✅ 1🔄 0❌ 0 issues, 0 comments 0
CLAassistant ⚡ 0 0✅ 0🔄 0❌ 0 issues, 50 comments 0

How is the score calculated? Read about it in the Reputation Bot repository 🤖

@CLAassistant
Copy link

CLAassistant commented Mar 2, 2026

CLA assistant check
All committers have signed the CLA.

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.

Cannot switch mcp server from OAuth to Authorization header

3 participants