Skip to content

fix(mcp): repair standalone OAuth login discovery - #3959

Draft
aheritier wants to merge 1 commit into
fix/3879-prm-helper-extractionfrom
fix/3879-debug-oauth-login-repair
Draft

fix(mcp): repair standalone OAuth login discovery#3959
aheritier wants to merge 1 commit into
fix/3879-prm-helper-extractionfrom
fix/3879-debug-oauth-login-repair

Conversation

@aheritier

Copy link
Copy Markdown
Collaborator

Stacked on #3955/#3881. This intentionally combines PR4a–4c because discovery, credentials, and token-key behavior are coupled.

Summary:

  • Forward the full latest.Remote.
  • Match names and URLs exactly.
  • Use authoritative challenge metadata, with path→root fallback only when metadata is absent.
  • Require configured-only credentials for explicit ID/secret (no DCR); use DCR with no prompt when there is no ID.
  • Preserve exact scopes across DCR, authorize, and RequestedScopes.
  • Use Remote.URL verbatim as the resource/key.
  • Add safe local httptest coverage.

Callback behavior is intentionally unchanged/deferred (PR4d/Q2). Documentation is planned in PR5.

No secrets, private test data, or logs are included.

@aheritier aheritier added area/mcp MCP protocol, MCP tool servers, integration kind/fix PR fixes a bug (maps to fix:). Use on PRs only. labels Aug 10, 2026
@aheritier
aheritier force-pushed the fix/3879-debug-oauth-login-repair branch from 4a3d3b0 to 03c6652 Compare August 10, 2026 20:28
@aheritier
aheritier force-pushed the fix/3879-debug-oauth-login-repair branch from 03c6652 to 5992a43 Compare August 10, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/mcp MCP protocol, MCP tool servers, integration kind/fix PR fixes a bug (maps to fix:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant