Skip to content

Conversation

mouraddmeiri
Copy link
Contributor

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.

@mouraddmeiri mouraddmeiri requested a review from a team as a code owner October 23, 2025 11:08
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nap-waf labels Oct 23, 2025
@github-actions
Copy link

Deploy Preview will be available once build job completes!

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

Copy link
Member

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

Generally LGTM! Left a comment about the Dockerfile.

Copy link
Member

Choose a reason for hiding this comment

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

This file is no longer in use.

See the new set of Dockerfiles in the folder /content/include/waf/dockerfiles/.

@ADubhlaoich ADubhlaoich added product/waf Issues related to F5 WAF for NGINX and removed product/nap-waf labels Oct 23, 2025
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/waf Issues related to F5 WAF for NGINX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants