RAOPST-4023: document the one-time first-sign-in consent dialog - #44
Open
OS-joaocastilho wants to merge 1 commit into
Open
RAOPST-4023: document the one-time first-sign-in consent dialog#44OS-joaocastilho wants to merge 1 commit into
OS-joaocastilho wants to merge 1 commit into
Conversation
The MCP server advertises the `openid offline_access` OAuth scope. On the first sign-in from any harness the tenant's IdP will show a one-time consent dialog before returning to the loopback callback, because `offline_access` is what most IdPs gate behind consent (it lets the sign-in survive an SSO-session boundary). It skips on every later sign-in on the same profile. The install recipes all say "opens a browser for OAuth sign-in" and leave the reader unprepared for the extra prompt. Add: - A short "Note on the first sign-in" section just above Troubleshooting, explaining what the dialog is, why it appears, and that it is one-time. - A Troubleshooting row that redirects to that note, so a user who searches the page for the dialog symptom lands on the explanation. Nothing on the server or in this repo can suppress the dialog, so the docs are the only lever. Companion to `OutSystems/rd-ai-ase-toolkit#223` (RAOPST-4023), which starts advertising the scope.
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.
Summary
Add a short note (and a Troubleshooting redirect) explaining the one-time consent dialog users will see on first OAuth sign-in against the OutSystems MCP server. Companion to
OutSystems/rd-ai-ase-toolkit#223(RAOPST-4023), which starts advertisingopenid offline_accesson all four OAuth channels.Problem / Why
rd-ai-ase-toolkit#223begins advertisingoffline_accesson the RFC 9728 protected-resource metadata, the RFC 8414 AS metadata, theWWW-Authenticatechallenge, and the RFC 7591 DCR registration response. Clients that see it request it; every tenant IdP we broker to shows a one-time consent screen the first time that scope is requested. Before this PR, every install recipe here says "opens a browser for OAuth sign-in" and leaves the reader unprepared for the extra prompt.No server-side change can suppress the dialog: it is emitted by the tenant IdP, not by this MCP or the AS proxy. The docs are the only lever.
How it is solved
Two pieces, both in
README.md:No install-recipe wording changes: they still say "opens a browser for OAuth sign-in", which stays accurate. The extra screen appears only once, so weaving it into every recipe would raise noise for everyone who has already installed once.
Alternatives considered
Why this option
Minimal footprint (five lines), single ground-truth explanation reachable from both entry points (top-down reader and troubleshooter), no coupling to individual harness sections. If the dialog wording or trigger changes upstream, only the one Note needs an edit.
Drawbacks & limitations
offline_accessby name to be honest about the mechanism. A non-technical reader may find the term unfamiliar; the surrounding sentence explains what it does in plain English, but a reader who scans only headings may not read that far.offline_accessbehind consent (Keycloak sometimes does not, depending on realm configuration), the Note is a false alarm for that user. Preferred over the reverse failure mode (silence when the dialog does appear).offline_access(via theOAUTH_SCOPES=nonekill switch in the server repo). The dialog would stop appearing but the Note would still be there. Cheap to remove if that ever ships.Smoke tests
Test plan
#note-on-the-first-sign-in) resolvesrd-ai-ase-toolkit#223is merged and image rolls out, so the dialog described here is actually reachable to a reader following the install path