Repository navigation
docs: add the RFC 9700 conformance matrix for the OAuth client - #129
Merged
Merged
Conversation
Checks every client recommendation against the API at v0.19.1 and lists the authorization server rows as applying from v0.21. Links it from the security posture page.
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.
Closes #117.
Adds
understand/oauth-rfc9700.mdxand links it from the security posture page (the OAuth threat row and Related Docs).v0.19.1tag (and@seamless-auth/typesv0.28.0for the request and provider schemas).Writing the matrix turned up one real gap: the OAuth
stateis not bound to the client that started the sign-in (login CSRF, RFC 9700 section 4.7). It is filed as fells-code/seamless-auth-api#463 and fixed in fells-code/seamless-auth-api#465 with fells-code/seamless-auth-types#104 and the SDK PR. The row says "Gap" with a caution note; once the fix is released the row should be updated to point at the release tag.Other rows that are not a clean pass, stated as such on the page: mix-up (no
isscheck, shared default redirect URI), client authentication (client_secret_postonly), sender-constrained tokens (not used), TLS on provider URLs (not enforced by the schema).Checks:
npm run lintclean,npm run typecheck0 errors,npm run build56 pages built (only the existingenvlanguage and empty i18n warnings).