Skip to content

fix: FP when being redirected to wp-admin on login - #114

Merged
azurit merged 6 commits into
coreruleset:masterfrom
azurit:FP
Oct 4, 2026
Merged

azurit merged 6 commits into
coreruleset:masterfrom
azurit:FP

Conversation

@azurit

@azurit azurit commented Oct 4, 2026

Copy link
Copy Markdown
Member

No description provided.

@EsadCetiner EsadCetiner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Your test also fires against rules 932236 and 942120, can you disable those too for ARGS:redirect_to?

@EsadCetiner

Copy link
Copy Markdown
Member

Also, can you change your PR title to something more descriptive?

@azurit

azurit commented Oct 4, 2026

Copy link
Copy Markdown
Member Author

Also, can you change your PR title to something more descriptive?

Feel free to change it.

@EsadCetiner EsadCetiner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't have enough information to write a good PR title, where did this false positive come from exactly? Only PR title I could suggest is fix: fp when being redirected to wp-admin on login

Comment thread tests/regression/wordpress-rule-exclusions-plugin/9507100.yaml
@azurit azurit changed the title fix: FP with 'redirect_to' parameter fix: FP when being redirected to wp-admin on login Oct 4, 2026
@azurit
azurit merged commit 679828d into coreruleset:master Oct 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants