-
Notifications
You must be signed in to change notification settings - Fork 435
docs: oauth2 stateless jwt access tokens #2368
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
base: master
Are you sure you want to change the base?
Conversation
vinckr
left a comment
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.
thanks, looks pretty good already although I do get very strong written by AI vibes 😁
it probably would fit here into the sidebar https://www.ory.com/docs/guides/token-management
left some style comments + also added upstream label since I assume this is for an upcoming feature
Yeah, and I had already trimmed some of the fat. 😁 I cleaned it up a bit more.
This feature is currently only meant for OEL, so is that sidebar location still applicable? Also, how can I add this to the sidebar?
Thank you! |
0e93007 to
6f689e2
Compare
You're right, we have a separate section for OEL only. |
unatasha8
left a comment
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.
Added some suggestions.
vinckr
left a comment
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.
nice 🐝
lgtm
let's wait until the feature is released before merging though.
Related Issue or Design Document
Add documentation around stateless JWT access token OEL feature
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact [email protected]) from the maintainers to push the changes.