[WAF] Improve managed rules troubleshooting: payload logging and malicious uploads - #31683
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
PR Review for #31683SummaryThis PR makes a clean, docs-only change to
The additions are well-placed, the links are valid, and no new components or frontmatter changes are required. Suggested Labels
Issues Flagged1. Minor technical inaccuracy — “encrypted with a key pair” Suggested change: Update the wording to match the existing documentation.
2. (Optional) Plan restriction formatting 3. (Optional) Link text for configuration page Verdict: LGTM with the suggested wording fix for the encryption description. All links verified, frontmatter unchanged, and no build concerns identified. |
Review
Code ReviewThis code review is in beta and may not always be helpful — use your judgment. No code review issues found. Conventions❌ This review could not complete this run; results may be incomplete. It will retry on the next push. Style Guide ReviewNo style-guide issues found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
Preview URL: https://579e6ec5.preview.developers.cloudflare.com Files with changes (up to 15) |
|
Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant). |
|
@elithrar @danielegm @hsaxenaCF - can you review this PR, please? Thanks |
|
Hey there, we've marked this pull request as stale because there's no recent activity on it. This label helps us identify PRs that might need updates (or to be closed out by our team if no longer relevant). |
|
@elithrar @danielegm - could you please review this PR? Thanks. |
|
/rebase |
5d42d8f to
6a3852a
Compare
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://zein-dee-3647-managed-rules-troubleshooting.previews.developers.cloudflare.com (commit 579e6ec)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
|
/rebase |
…ules troubleshooting DEE-3647
6a3852a to
579e6ec
Compare
Resolves DEE-3647.
Adds two pieces of guidance to the WAF managed rules troubleshooting page that were missing from the page:
Intentionally not changed
The ticket also suggested adding WAF attack score guidance for false negatives and clarifying exception-vs-disable behavior. Both are already documented on this page (the Troubleshoot false negatives section already recommends WAF attack score, and the Troubleshoot false positives section already covers exceptions vs disabling rules), so they were left as-is to avoid duplication.
Notes