Skip to content

NGINXaaS updates to use F5 WAF for NGINX - #2232

Merged
JTorreG merged 1 commit into
mainfrom
nginxaas-2343
Sep 3, 2026
Merged

NGINXaaS updates to use F5 WAF for NGINX#2232
JTorreG merged 1 commit into
mainfrom
nginxaas-2343

Conversation

@edarzins

Copy link
Copy Markdown
Contributor

Document usage and limitations of F5 WAF for NGINX when used with NGINXaaS (Google and AWS).

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.

@edarzins
edarzins requested a lite review from Copilot August 27, 2026 22:27
@edarzins
edarzins requested a review from a team as a code owner August 27, 2026 22:27
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 27, 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/2232/

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the NGINXaaS documentation set to describe how to enable/disable and configure F5 WAF for NGINX for NGINXaaS for Google Cloud and NGINXaaS for AWS, and aligns AWS docs/changelog with the new (Preview) support and logging guidance.

Changes:

  • Adds new provider-specific “Enable”, “Configure”, and “Disable” F5 WAF for NGINX how-to pages for Google Cloud and AWS.
  • Updates AWS deployment-creation docs and AWS log-export docs to include WAF enablement and WAF security log setup.
  • Updates the NGINXaaS changelog and removes the outdated “WAF not supported” constraint from the AWS overview.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
content/nginxaas/google/app-protect/_index.md Adds a new Google Cloud “F5 WAF for NGINX” section index page.
content/nginxaas/google/app-protect/enable-waf.md New how-to for enabling WAF on Google Cloud deployments.
content/nginxaas/google/app-protect/configure-waf.md New how-to for configuring WAF directives/policy on Google Cloud.
content/nginxaas/google/app-protect/disable-waf.md New how-to for disabling WAF on Google Cloud deployments.
content/nginxaas/aws/app-protect/_index.md Adds a new AWS “F5 WAF for NGINX” section index page.
content/nginxaas/aws/app-protect/enable-waf.md New how-to for enabling WAF on AWS deployments.
content/nginxaas/aws/app-protect/configure-waf.md New how-to for configuring WAF directives/policy on AWS.
content/nginxaas/aws/app-protect/disable-waf.md New how-to for disabling WAF on AWS deployments.
content/nginxaas/changelog.md Adds an August 26, 2026 entry announcing AWS WAF support (Preview).
content/nginxaas/aws/overview.md Removes the outdated AWS constraint stating WAF is not supported.
content/nginxaas/aws/monitoring/enable-nginx-logs.md Adds a section to configure/export WAF security logs.
content/nginxaas/aws/deploy/create-deployment/deploy-console.md Adds WAF enablement to create/edit deployment console steps.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread content/nginxaas/aws/app-protect/enable-waf.md Outdated
Comment thread content/nginxaas/aws/app-protect/enable-waf.md
Comment thread content/nginxaas/google/app-protect/enable-waf.md
Comment thread content/nginxaas/aws/app-protect/disable-waf.md
Comment thread content/nginxaas/google/app-protect/disable-waf.md
Comment thread content/nginxaas/google/app-protect/configure-waf.md
Comment thread content/nginxaas/changelog.md
Comment thread content/nginxaas/aws/deploy/create-deployment/deploy-console.md
Comment thread content/nginxaas/aws/deploy/create-deployment/deploy-console.md Outdated
Comment thread content/nginxaas/aws/monitoring/enable-nginx-logs.md Outdated
@edarzins
edarzins requested a review from JTorreG August 28, 2026 14:32
Comment thread content/nginxaas/aws/app-protect/_index.md Outdated
Comment thread content/nginxaas/aws/app-protect/_index.md Outdated
Comment thread content/nginxaas/aws/app-protect/configure-waf.md Outdated

@amudukutore amudukutore left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I've listed some minor comments but the rest of the content looks good to me. Approving these changes (assuming we can discuss/address those comments).

@edarzins

edarzins commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

@JTorreG Pending on review from nginx-docs

Comment thread content/nginxaas/aws/deploy/create-deployment/deploy-console.md Outdated
Comment thread content/nginxaas/overview/app-protect/configure-waf.md Outdated

@JTorreG JTorreG left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The new docs are missing a few of the frontmatter metadata values (description, f5-summary, f5-keywords... )

Please review the templates in the https://github.com/nginx/documentation/tree/main/templates folder for guidance on these fields

Comment thread content/nginxaas/overview/app-protect/configure-waf.md
Comment thread content/nginxaas/overview/app-protect/disable-waf.md
Comment thread content/nginxaas/overview/app-protect/enable-waf.md
@edarzins

edarzins commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

@JTorreG All suggestions accepted, thanks!

Document usage and limitations of F5 WAF for NGINX when used with
NGINXaaS (Google and AWS).
@JTorreG
JTorreG merged commit 034238d into main Sep 3, 2026
9 checks passed
@JTorreG
JTorreG deleted the nginxaas-2343 branch September 3, 2026 15:13
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants