Skip to content

[Rules] Fix Cache Rules changelog bot field list - #32802

Open
damiete wants to merge 2 commits into
cloudflare:productionfrom
damiete:fix/cache-rules-changelog-bot-fields
Open

[Rules] Fix Cache Rules changelog bot field list#32802
damiete wants to merge 2 commits into
cloudflare:productionfrom
damiete:fix/cache-rules-changelog-bot-fields

Conversation

@damiete

@damiete damiete commented Aug 17, 2026

Copy link
Copy Markdown

What

Correct the 2026-07-16 Cache Rules changelog field table.

Why

Three listed fields are not valid Rules language fields:

  • cf.bot_management.api_score — does not exist
  • cf.bot_management.attack_score — does not exist (WAF attack score is cf.waf.score)
  • cf.bot_management.bot_tags["<TAG>"] — does not exist (and bot tags are documented as log-only today)

Also list Bot Management fields that are already documented and available in rules: detection_ids, signed_agent.

Changes

Page

  • /changelog/2026-07-16-cache-rules-bot-fields-asn/

@damiete
damiete requested review from a team as code owners August 17, 2026 17:02
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No issues found in commit 848c9c9.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

No convention issues found.

Style Guide Review

No style-guide issues found.

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.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.
/rebase Rebases the PR branch against production. On conflict, attempts to resolve automatically using AI. Stops with an explanation if confidence is not high enough.

Remove non-existent api_score and attack_score fields, correct
bot_tags to tags, and list detection_ids and signed_agent.
@damiete
damiete force-pushed the fix/cache-rules-changelog-bot-fields branch from 5c79215 to 6c2b6bb Compare August 17, 2026 17:15
Bot tags remain log-only in public docs; do not list
cf.bot_management.tags as a Cache Rules expression field.
@damiete

damiete commented Aug 17, 2026

Copy link
Copy Markdown
Author

Review triage

Finding Decision
CR-5b0b902262ef · cf.bot_management.tags Agree — removed the row. Public bot-tags docs still say tags are log-only (BotTags), and there is no field-reference page for cf.bot_management.tags. Keeping it would reintroduce the same class of error this PR is fixing.

Fix pushed in 848c9c95. PR still removes the three non-existent fields (api_score, attack_score, bot_tags[…]) and keeps detection_ids / signed_agent (already in Bot Management variables).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant