-
Notifications
You must be signed in to change notification settings - Fork 25.2k
update JWT realm docs to include at+jwt support #126841
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update JWT realm docs to include at+jwt support #126841
Conversation
Documentation preview: |
Pinging @elastic/es-docs (Team:Docs) |
Pinging @elastic/es-security (Team:Security) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM -- optional nit
@@ -294,7 +294,7 @@ token. | |||
as `HS256`. The algorithm must be in the realm's allow list. | |||
|
|||
`typ`:: | |||
(Optional, String) Indicates the token type, which must be `JWT`. | |||
(Optional, String) Indicates the token type. For an ID token, this must be `JWT`; for access tokens, this must be `JWT` or `at+jwt`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: unless its a pain because of asciidoc formatting, lets match the itemized format from: https://github.com/elastic/docs-content/pull/1124/files#diff-3761114bd79abf4acad1c5d598f9cbafc116580eeb95beeda3dc0f1466fdd5fcR221
7d9577a
to
d98662a
Compare
Update docs to mention
at+jwt
support introduced hereSee also elastic/docs-content#1124