Skip to content

fix same origin for protected resource metadata #8410

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.

Sign up for GitHub

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

Jump to bottom
Merged
pjain1 merged 6 commits into from
Nov 28, 2025
Merged

fix same origin for protected resource metadata #8410

pjain1 merged 6 commits into from
Nov 28, 2025

Conversation

@pjain1
Copy link
Member

@pjain1 pjain1 commented Nov 28, 2025 •
edited
Loading

When connecting to MCP server using Claude code terminal it throws error if api is used instead of admin in the url. Interestingly it works fine for Claude Desktop and ChatGPT Web app.

[5341] Connection error: Error: Protected resource https://admin.rilldata.in does not match expected https://api.rilldata.in/v1/organizations/<>/projects/<>/runtime/mcp (or origin)

Still using admin.rilldata for auth server and registration url returned from auth server metadata call.

Also added docs for connecting from Claude code.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!
begelundmuller
begelundmuller reviewed Nov 28, 2025
View reviewed changes
@pjain1 pjain1 merged commit 8bf99bd into main Nov 28, 2025
13 of 16 checks passed
@pjain1 pjain1 deleted the admin_api_mcp_compatibility branch November 28, 2025 15:04
pjain1 added a commit that referenced this pull request Nov 28, 2025
@pjain1
fix same origin for protected resource metadata (#8410)
6af9401 
* fix same origin for protected resource metadata

* spelling

* look at request to decide replacement

* comment and url

* return same resource url as requested resource

* fix test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@pjain1 @begelundmuller