Skip to content

Conversation

@migueldemoura
Copy link
Contributor

These actions have either never been used or are obsolete and about to be removed.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 5, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 339a3f8
Status: ✅  Deploy successful!
Preview URL: https://39f5e806.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://mdemoura-remove-obsolete-cha.cloudflare-docs-7ou.pages.dev

View logs

@pedrosousa
Copy link
Contributor

Hi @migueldemoura
Do you have any suggestions regarding the WAF FAQ entries where we mention these challenge actions?
#1, #2, #3

@migueldemoura
Copy link
Contributor Author

Happy to adjust those, but the tl;dr is that they were never useful and just outright misleading as we never log the overwhelming majority of challenge failures to security events.

@pedrosousa
Copy link
Contributor

We also have a couple of occurrences in the following page, which we can probably delete if they no longer make sense:
https://github.com/cloudflare/cloudflare-docs/blob/production/src/content/docs/logs/reference/security-fields.mdx?plain=1

These actions have either never been used or are obsolete and about to
be removed.
@migueldemoura migueldemoura force-pushed the mdemoura/remove-obsolete-challenge-actions branch from 0920432 to 339a3f8 Compare December 5, 2024 14:56
@migueldemoura migueldemoura requested review from a team and pedrosousa as code owners December 5, 2024 14:56
@migueldemoura
Copy link
Contributor Author

Made a few adjustments, feel free to change them! I also noticed that we don't mention every kind of challenge type when referring to the action names, but I guess that can be fixed in a follow-up.

@pedrosousa pedrosousa merged commit 65e9289 into production Dec 6, 2024
12 checks passed
@pedrosousa pedrosousa deleted the mdemoura/remove-obsolete-challenge-actions branch December 6, 2024 09:42
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.

9 participants