Uh oh!
There was an error while loading. Please reload this page.
docs: API key shown-once behavior + permission scoping (155917/155921/156737) - #146
Open
mhmojtahedi wants to merge 2 commits into
Open
docs: API key shown-once behavior + permission scoping (155917/155921/156737)#146mhmojtahedi wants to merge 2 commits into
mhmojtahedi wants to merge 2 commits into
Conversation
… scoping - Authentication page: the private key is displayed only at creation and cannot be re-read; keys are full-access (grandfathered) or scoped to an explicit permission set. - New reference page: the operation-to-permission catalog for scoped keys, in the same vocabulary as dashboard users. - API reference enrichment: the shared API-Key permission blocks no longer claim unconditional admin access; they point scoped-key holders at the catalog. Backend tickets: #155917 #155921 #156737 (core PRs #208, #350).
…API reference - Business how-to no longer tells merchants to re-open the key later: the private key is copied at creation or never; the entry keeps only the public key. Form table gains Full access and Permissions fields. (Screenshots 03-05 show the old admin and need retakes.) - npm run gen-api: every generated endpoint page's Permissions table now describes full-access vs scoped keys instead of 'All permissions (admin access)'.
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 freeto 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.
Documents the API-key security work from core_backend tickets #155917, #155921, #156737 (core PRs ottuco/core_backend#208, ottuco/core_backend#350).
What changed
Developers
getting-started/authentication.md— the private key is now shown once at creation and cannot be re-read; keys are full access (all pre-existing keys, unchanged) or scoped to an explicit permission set. Removed the "all permissions are granted by default" claim.reference/api-key-permissions.md— the operation→permission catalog for scoped keys (same vocabulary as dashboard users), with the guarantees stated up front: existing keys unaffected, SDK/public-key guest flows never blocked, scoped-without-perm → 403. Added to the Reference sidebar.Business
settings/api-keys.mdx— the step-by-step no longer instructs merchants to click the key later to view it (impossible now). Step 4 is "copy the private key — shown only once"; the entry keeps only the public key. Form table gains Full access and Permissions fields.API reference (generated)
_shared/permissions.yamlblocks updated, thennpm run gen-api— every endpoint page's Permissions table now describes both key modes and links to the catalog instead of claiming "All permissions (admin access)".Review notes
api-keys-03/04/05show the old admin form (no Full access/Permissions fields, old key-viewing flow) and need retakes once a server with the new admin is available.npm run buildpasses; the broken-anchor warnings in the log are pre-existing onmain(glossary term anchors etc.), none from these pages.