Skip to content

Add JWT authentication page for NGF#1774

Merged
dhurley merged 5 commits intongf-release-2.5from
ngf/add-jwt-auth
Mar 27, 2026
Merged

Add JWT authentication page for NGF#1774
dhurley merged 5 commits intongf-release-2.5from
ngf/add-jwt-auth

Conversation

@dhurley
Copy link
Copy Markdown
Contributor

@dhurley dhurley commented Mar 10, 2026

Proposed changes

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@dhurley dhurley requested a review from a team as a code owner March 10, 2026 10:47
@github-actions github-actions Bot added documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric labels Mar 10, 2026
@github-actions
Copy link
Copy Markdown

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/1774/

Comment thread content/ngf/traffic-security/jwt-authentication.md
Comment thread content/ngf/traffic-security/jwt-authentication.md Outdated
Comment thread content/ngf/traffic-security/jwt-authentication.md Outdated
Copy link
Copy Markdown
Contributor

@salonichf5 salonichf5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall looks pretty good to me, just a couple of nits

Comment thread content/ngf/traffic-security/jwt-authentication.md Outdated
Comment thread content/ngf/traffic-security/jwt-authentication.md
@dhurley dhurley requested a review from a team as a code owner March 25, 2026 16:23
Comment thread content/ngf/traffic-security/jwt-authentication.md
Comment thread content/ngf/traffic-security/jwt-authentication.md Outdated
Comment thread content/ngf/traffic-security/jwt-authentication.md
Comment thread content/ngf/traffic-security/jwt-authentication.md
Comment thread content/ngf/traffic-security/jwt-authentication.md Outdated
Copy link
Copy Markdown
Contributor

@salonichf5 salonichf5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall looks good, just a couple of recommendations

Copy link
Copy Markdown
Contributor

@salonichf5 salonichf5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved by mistake

Copy link
Copy Markdown
Contributor

@salonichf5 salonichf5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

Comment thread content/includes/ngf/keycloak-certs.md Outdated
@dhurley dhurley merged commit ce123d9 into ngf-release-2.5 Mar 27, 2026
6 checks passed
@dhurley dhurley deleted the ngf/add-jwt-auth branch March 27, 2026 13:55
sjberman pushed a commit that referenced this pull request Mar 27, 2026
sjberman pushed a commit that referenced this pull request Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants