Skip to content

Fix Descope auth flow in remote-mcp-server-descope-auth demo#607

Open
mariamantsurova wants to merge 1 commit into
cloudflare:mainfrom
mariamantsurova:descope-inbound-app-fixes
Open

Fix Descope auth flow in remote-mcp-server-descope-auth demo#607
mariamantsurova wants to merge 1 commit into
cloudflare:mainfrom
mariamantsurova:descope-inbound-app-fixes

Conversation

@mariamantsurova

Copy link
Copy Markdown

Align the demo with Descope Inbound Apps and the modern Workers/MCP setup:

  • Use an Inbound App Client ID/Secret instead of the Project ID + Management Key. Rename env vars to DESCOPE_CLIENT_ID / DESCOPE_CLIENT_SECRET.
  • Token exchange now uses the confidential-client form (application/ x-www-form-urlencoded with client_id + client_secret) against /oauth2/v1/apps/token.
  • Request openid profile email scopes on /authorize so /userinfo works.
  • Add nodejs_compat compatibility flag so the Worker builds.
  • Update README + landing page: Inbound App creation via Agentic Identity Hub, user-information scope mapping, Streamable HTTP (/mcp) instead of SSE, corrected wrangler kv commands, and a troubleshooting table.
  • Refresh Inspector screenshot.

@changeset-bot

changeset-bot Bot commented Jul 8, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 578fe23

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mariamantsurova mariamantsurova force-pushed the descope-inbound-app-fixes branch from b629bc9 to 578fe23 Compare July 8, 2026 11:12
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.

1 participant