Skip to content
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

Docs: Fix link in security.md #7007

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Docs: Fix link in security.md #7007

merged 1 commit into from
Dec 17, 2024

Conversation

nginx-aoife
Copy link
Contributor

Broken link found in kubernetes-ingress by Docs Nightly Automation Testing

Fixing a broken link to WAFv5 docs.

Also making the "incompatible link" more specific to v4 to avoid confusion. Previously it was bringing user to top level WAF which included v4 & v5.

Please Merge and Cherry Pick into release branch if approved.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Fixing a broken link to WAFv5 docs.
Making the incompatible link more specific to v4 to avoid confusion.

Signed-off-by: nginx-aoife <[email protected]>
@nginx-aoife nginx-aoife requested review from a team as code owners December 17, 2024 10:02
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Dec 17, 2024
@nginx-aoife nginx-aoife changed the title Fix link in security.md Docs: Fix link in security.md Dec 17, 2024
@jjngx jjngx merged commit d372e25 into nginxinc:main Dec 17, 2024
30 checks passed
AlexFenlon pushed a commit that referenced this pull request Dec 17, 2024
AlexFenlon added a commit that referenced this pull request Dec 17, 2024
Docs: Fix link in security.md (#7007)

Co-authored-by: nginx-aoife <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

3 participants