Skip to content

feat: wire OAuth endpoint discovery into gateway UI and service (#1435)#3571

Open
WilliamChen000 wants to merge 4 commits intoIBM:mainfrom
WilliamChen000:feat/1435-oauth-discover-ui
Open

feat: wire OAuth endpoint discovery into gateway UI and service (#1435)#3571
WilliamChen000 wants to merge 4 commits intoIBM:mainfrom
WilliamChen000:feat/1435-oauth-discover-ui

Conversation

@WilliamChen000
Copy link
Copy Markdown

🔗 Related Issue

Closes #1435


📝 Summary

Wires the OAuth endpoint discovery feature from #1435 end-to-end.


🏷️ Type of Change

  • admin.py: Add POST /admin/gateways/discover-oauth endpoint that calls DcrService.discover_as_metadata() and returns discovered endpoints
  • gateway_service.py: Auto-trigger discovery on gateway create/update when issuer is set but token_url/authorization_url are missing
  • admin.html: Add discoverOAuthEndpoints() JS function for the existing Discover buttons — populates fields, marks them read-only, shows DCR badge and success/error status

🧪 Verification

  • Tested with mock OAuth server at http://localhost:3003
  • Discover button successfully populates Token URL and Authorization URL
  • DCR available badge shows correctly
  • Error case shows clear message prompting manual configuration
  • Passed Make lint and Make test

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • Tests added/updated for changes
  • Documentation updated (if applicable)
  • No secrets or credentials committed

📓 Notes (optional)

Screenshots, design decisions, or additional context.

…1435)

Signed-off-by: root <root@yy.localdomain>
Signed-off-by: WilliamChen000 <chenwi@tcd.ie>
Signed-off-by: WilliamChen000 <chenwi@tcd.ie>
@WilliamChen000 WilliamChen000 force-pushed the feat/1435-oauth-discover-ui branch from fa78da3 to 7263bc4 Compare March 10, 2026 02:26
@crivetimihai crivetimihai added enhancement New feature or request ui User Interface security Improves security labels Mar 14, 2026
@crivetimihai crivetimihai added this to the Release 1.1.0 milestone Mar 14, 2026
@crivetimihai
Copy link
Copy Markdown
Member

Thanks @WilliamChen000 — OAuth endpoint discovery wired into the gateway UI (#1435). Targeting 1.1.0.

@crivetimihai crivetimihai added the SHOULD P2: Important but not vital; high-value items that are not crucial for the immediate release label Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request security Improves security SHOULD P2: Important but not vital; high-value items that are not crucial for the immediate release ui User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE][AUTH]: Infer identity provider info for onboarded MCP servers

2 participants