Skip to content

[API Shield] Update API Shield phase definition in rulesets-engine phase list - #33697

Merged
djhworld merged 1 commit into
cloudflare:productionfrom
djhworld:cloudflare-docs-APISHI-5818
Sep 25, 2026
Merged

djhworld merged 1 commit into
cloudflare:productionfrom
djhworld:cloudflare-docs-APISHI-5818

Conversation

@djhworld

@djhworld djhworld commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Updates the API Shield phase definition in rulesets-engine phase list to reference these phases as internal

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

AI Review

✅ Reviewed e7674b5 · 0 findings · incremental

Resolved (1) · Dismissed (0)
  • Resolved: PR description is empty · PR description at head is no longer empty — it briefly states the change (updates API Shield phase definitions in the rulesets-engine phase list to mark them internal), which passes the conventions rule for a brief but clear description.
Commands
  • /review Run a review now.
  • /full-review Review the entire PR diff.
  • /disable-auto-review Stop automatic reviews.
  • /rebase Rebase against production.

@janrueth janrueth left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we wanted to explicitly call out that one is used for detection and the other for mitigation otherwise customers can't know when something is available (after detection) and when something blocks compared to other things

@djhworld
djhworld force-pushed the cloudflare-docs-APISHI-5818 branch from d1fa966 to e7674b5 Compare September 25, 2026 11:56
@djhworld

Copy link
Copy Markdown
Collaborator Author

I think we wanted to explicitly call out that one is used for detection and the other for mitigation otherwise customers can't know when something is available (after detection) and when something blocks compared to other things

fixed to add the detection/mitigation split + added new footnotes at the bottom

@djhworld
djhworld merged commit 51904dd into cloudflare:production Sep 25, 2026
15 checks passed
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.

5 participants