Skip to content

Conversation

ADubhlaoich
Copy link
Contributor

@ADubhlaoich ADubhlaoich commented Oct 3, 2025

Proposed changes

This commit adds a handful of pages to F5 WAF for NGINX, migrating and rewriting content from the prior documentation set.

It includes:

  • Custom dimensions for log entries
  • Directives
  • Do-nothing
  • Override rules
  • User-defined URLs and parameters
  • Violations

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.

This commit adds a handful of pages to F5 WAF for NGINX, migrating and
rewriting content from the prior documentation set.

It includes:

- Custom dimensions for log entries
- User-defined URLs and parameters
- Do-nothing
- Override rules
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 3, 2025
@ADubhlaoich
Copy link
Contributor Author

ADubhlaoich commented Oct 3, 2025

Content is largely complete in the sense that it exists, but I need to rewrite the source content for the Custom dimensions and Override rules documents.

Should be finished by lunch time on Monday.

Edit: Added Directives and External references pages, doesn't change my ETA.

These three pages are grouped with Violations and the Policy parameter reference pages, ordered before the individual feature pages as they are conceptual knowledge that applies to all policies.

Copy link

github-actions bot commented Oct 3, 2025

Deploy Preview will be available once build job completes!

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

@ADubhlaoich ADubhlaoich self-assigned this Oct 7, 2025
@ADubhlaoich ADubhlaoich marked this pull request as ready for review October 7, 2025 14:08
@ADubhlaoich ADubhlaoich requested a review from a team as a code owner October 7, 2025 14:08
Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

I have non-blocking comments that I don't feel strongly about

Copy link
Contributor

@JTorreG JTorreG left a comment

Choose a reason for hiding this comment

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

approved with suggestions

@ADubhlaoich ADubhlaoich merged commit a21189d into main Oct 7, 2025
8 checks passed
@ADubhlaoich ADubhlaoich deleted the waf/port-more-features branch October 7, 2025 15:01
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.

3 participants