Skip to content

Initial WAF support for NGINXaaS for Google#1939

Merged
edarzins merged 3 commits into
mainfrom
waf-phase1
May 14, 2026
Merged

Initial WAF support for NGINXaaS for Google#1939
edarzins merged 3 commits into
mainfrom
waf-phase1

Conversation

@edarzins

@edarzins edarzins commented May 7, 2026

Copy link
Copy Markdown
Contributor

Support for enabling WAF per deployment, NGINX configurations with WAF, and WAF logging.

Proposed changes

Checklist

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

@richbrowne @JTorreG

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 review from a team as code owners May 7, 2026 17:35
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 7, 2026
@github-actions

github-actions Bot commented May 7, 2026

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/1939/

@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.

use the ref shortcode for links to docs.nginx.com

Comment thread content/nginxaas-google/getting-started/create-deployment/deploy-console.md Outdated
Comment thread content/includes/nginxaas-google/logging-config-security-logs.md Outdated
Comment thread content/nginxaas-google/getting-started/nginx-configuration/overview.md Outdated
Comment thread content/nginxaas-google/getting-started/nginx-configuration/overview.md Outdated
Comment thread content/nginxaas-google/getting-started/nginx-configuration/overview.md Outdated
Comment thread content/nginxaas-google/changelog.md Outdated
Comment thread content/nginxaas-google/changelog.md Outdated

@richbrowne richbrowne 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.

Ok, I reviewed and the changes look good.

richbrowne

This comment was marked as low quality.

@edarzins edarzins force-pushed the waf-phase1 branch 2 times, most recently from c60b8d9 to 39cfa2f Compare May 13, 2026 19:35
@edarzins

Copy link
Copy Markdown
Contributor Author

@amudukutore @JTorreG Ready for re-review and approval.

Support for enabling WAF per deployment, NGINX configurations with WAF,
and WAF logging.
Comment thread content/includes/nginxaas-google/logging-config-security-logs.md Outdated
Comment thread content/includes/nginxaas-google/logging-config-security-logs.md Outdated
edarzins and others added 2 commits May 14, 2026 10:00
Co-authored-by: Puneet Singh Sarna <p.sarna@f5.com>
Co-authored-by: Puneet Singh Sarna <p.sarna@f5.com>
@edarzins edarzins merged commit dff609f into main May 14, 2026
9 checks passed
@edarzins edarzins deleted the waf-phase1 branch May 14, 2026 16:08
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.

6 participants