Repository navigation
fix(mcp-auth): repoint IAS public-PKCE OAuth to new tenant alzmza7li (#2550) - #2559
Merged
Merged
Conversation
…7li (#2550) - dev.mtaext XSUAA_MCP_URL: atxgsg7zi -> alzmza7li (fresh SAP Universal ID trust from #2551; old tenant retired at the subaccount trust layer). - Pinned dev client_id d01a9789-... -> 02782268-... (tutorials-identity recreated on alzmza7li; broker minted a fresh secretless public client, credential-type:NONE key verified NO secret/cert). - Loopback-only redirect-uris on the new instance: IAS rejected the carried- over non-loopback wildcards (*.cfapps.*, vscode://); public-PKCE MCP only needs http://localhost:*|127.0.0.1:* callbacks. - Sync mta-mcp.yaml comments + well-known-oauth IAS fixture to new client/tenant. Refs #2550, #2551
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Repoints the MCP public-PKCE OAuth (IAS) discovery + client from the retired tenant
atxgsg7zito the fresh SAP Universal ID tenantalzmza7li. This finishes the OAuth-mechanics work merged in #2544/#2545, which was blocked (#2551) on the old tenant's broken SAP-ID SAML trust. The subaccount trust is already cut over toalzmza7li(sap.customorigin = "Tutorial System (SAP Universal ID)", direct Universal ID); only the MCP service instance + approuter discovery env lagged behind.Changes
deploy/dev.mtaext— approuterXSUAA_MCP_URL:atxgsg7zi→alzmza7li.accounts.ondemand.com. This is the issuer mcp-remote reads from.well-known/oauth-authorization-server; it still served the dead tenant (verified live viacf env).tutorials-identityCF instance onalzmza7li(out-of-band, as theexisting-serviceit's adopted as). Broker minted a fresh secretless public client02782268-9593-409a-8481-701bfa44ebc7;credential-type:NONEkey verified NO clientsecret / NO cert. Loopback-only redirect-uris (IAS rejects the carried-over*.cfapps.*double-wildcard +vscode://scheme; public-PKCE MCP only needshttp://localhost:*/127.0.0.1:*callbacks).docs/end-users/mcp-quickstart.md— pinned devclient_id→ the new UUID.mta-mcp.yaml+test/unit/well-known-oauth.test.js— sync comments / IAS fixture to the new client + tenant.Deploy state (DEV)
tutorials-mcpMTA already redeployed →tutorials-srv-mcprebound to the recreatedtutorials-identity(bind confirmed).XSUAA_MCP_URL) runs from fresh DEV after this merges — the deploy guard requires HEAD == origin/DEV.Verification (after merge + deploy)
.well-known/oauth-authorization-serveradvertisesalzmza7liauthorize/token endpoints,openidscope,noneauth method.mcp-remotelogin asI809764completes WITHOUT the 400 that blocked IAS tenant reset — recreate shared PROD/DEV trust safely (unblocks #2550) #2551.resolveUserSapIdIAS-claim branch is a follow-up — needs a real federated token to map the I-number claim; deferred to a subsequent PR).Refs #2550, #2551.
🤖 Generated with Claude Code