-
Notifications
You must be signed in to change notification settings - Fork 167
Improved JWKs Validation PR #7064
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
Improved JWKs Validation PR #7064
Conversation
|
|
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
✅ PS. Add to the end of url /docs/nightly
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ PS. Add to the end of url /docs/nightly
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Some suggested tweaks to the content
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-authorization.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-claim-validation.md
Outdated
Show resolved
Hide resolved
tyk-docs/content/api-management/authentication/jwt-claim-validation.md
Outdated
Show resolved
Hide resolved
...s/content/basic-config-and-security/security/authentication-authorization/json-web-tokens.md
Outdated
Show resolved
Hide resolved
Co-authored-by: andyo-tyk <99968932+andyo-tyk@users.noreply.github.com>
…tion-authorization/json-web-tokens.md Co-authored-by: andyo-tyk <99968932+andyo-tyk@users.noreply.github.com>
…ation.md Co-authored-by: andyo-tyk <99968932+andyo-tyk@users.noreply.github.com>
…ation.md Co-authored-by: andyo-tyk <99968932+andyo-tyk@users.noreply.github.com>
User description
Contributor checklist
New Contributors
masterPR Type
Documentation
Description
Add dedicated JWT claim validation guide
Extract claim validation from JWT overview
Introduce JWT docs menu restructuring
Clarify configs, examples, and notes
Diagram Walkthrough
File Walkthrough
jwt-claim-validation.md
New comprehensive JWT claim validation guidetyk-docs/content/api-management/authentication/jwt-claim-validation.md
json-web-tokens.md
Extract claim validation from JWT overviewtyk-docs/content/basic-config-and-security/security/authentication-authorization/json-web-tokens.md
menu.yaml
Restructure JWT navigation menutyk-docs/data/menu.yaml