Skip to content

[WAF] Add threat intelligence detection docs#31205

Open
hsaxenaCF wants to merge 2 commits into
productionfrom
hsaxena/threat-intel-waf-docs
Open

[WAF] Add threat intelligence detection docs#31205
hsaxenaCF wants to merge 2 commits into
productionfrom
hsaxena/threat-intel-waf-docs

Conversation

@hsaxenaCF
Copy link
Copy Markdown
Collaborator

Summary

New documentation section at /waf/detections/threat-intelligence/ for the threat intelligence WAF detection (SHIP-11629). Four pages:

  • Overview (index.mdx) — what the detection does, data freshness (seven-day window), availability (Cloudforce One subscription)
  • Get started (get-started.mdx) — create a rule in Log mode, review matches in Security Analytics, switch to Block/Challenge, optional Saved Views export
  • Available fields (fields.mdx) — five cf.intel.ip.* fields, case sensitivity, union/flattened matching behavior, analytics gap (rule fields vs logged fields)
  • Example rules (example-rules.mdx) — six recipes covering Log, Block, Challenge, rate limiting, and combining with attack score

Documentation checklist

  • Is there a changelog entry? Separate PR: see the companion changelog PR.
  • The change adheres to the documentation style guide.
  • If a larger change — an issue has been opened. N/A — new section for a new feature.
  • Files which have changed name or location have redirects. N/A — all new files.

New section at /waf/detections/threat-intelligence/ with four pages:
overview (concept), get started, available fields (reference), and
example rules (configuration). Requires active Cloudforce One
subscription. Fields match on client IP address against a seven-day
threat intelligence database.
@hsaxenaCF hsaxenaCF requested review from a team and elithrar as code owners June 3, 2026 12:12
@cloudflare-docs-bot
Copy link
Copy Markdown

cloudflare-docs-bot Bot commented Jun 3, 2026

Review

✅ No style-guide issues found in commit 0349b6a.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/waf/ @pedrosousa, @cloudflare/firewall, @cloudflare/appsec-reviewers, @elithrar, @cloudflare/product-owners, @hsaxenaCF

@ask-bonk ask-bonk Bot added content:new Request for new/missing content triage Waiting for a docs team review. labels Jun 3, 2026
Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

Reviewed. Applied product:waf, content:new, and triage labels. One high-severity issue and one minor style item need attention.

Comment thread src/content/docs/waf/detections/threat-intelligence/index.mdx Outdated
Comment thread src/content/docs/waf/detections/threat-intelligence/get-started.mdx Outdated
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Jun 3, 2026

Reviewed PR #31205 and posted two inline comments.

Labels applied: product:waf, content:new, triage

Issues found:

  • HIGHget-started.mdx line 53: The curl example uses PUT /zones/{zone_id}/rulesets/phases/http_request_firewall_custom/entrypoint, which replaces all existing custom rules with a single rule. This is risky for a get-started guide. I recommended linking to Create a custom rule via API instead, which is consistent with other WAF detection get-started pages.

  • LOWindex.mdx line 15: Uses time-bound wording ("Today,") which the style guide advises avoiding. Posted a suggestion to remove it.

The frontmatter, component imports, link formatting, and structure are otherwise consistent with existing WAF detection docs.

github run

- Remove 'Today,' from index.mdx intro per style guide
- Replace raw PUT curl (which replaces all rules) with a link to
  'Create a custom rule via API' plus the expression example,
  matching the pattern used by sibling detection get-started pages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content:new Request for new/missing content product:waf size/m triage Waiting for a docs team review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants