Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FP 932260: Name of "Axel" #3644

Closed
ssigwart opened this issue Apr 3, 2024 · 5 comments · Fixed by #3705
Closed

FP 932260: Name of "Axel" #3644

ssigwart opened this issue Apr 3, 2024 · 5 comments · Fixed by #3705
Assignees

Comments

@ssigwart
Copy link
Contributor

ssigwart commented Apr 3, 2024

Description

This is very similar to #3490, but with a fill name field. For example, "Axel Smith" gets blocked.

How to reproduce the misbehavior (-> curl call)

curl -i -H "x-format-output: txt-matched-rules" 'https://sandbox.coreruleset.org?name=axel+smith'
Date: Wed, 03 Apr 2024 15:41:16 GMT
Content-Type: text/plain
Transfer-Encoding: chunked
Connection: keep-alive
X-Unique-ID: Zg14nKzdFlh2Fu_y9nB4ewAAAIg
x-backend: apache-nightly

932260 PL1 Remote Command Execution: Direct Unix Command Execution
949110 PL1 Inbound Anomaly Score Exceeded (Total Score: 5)
980170 PL1 Anomaly Scores: (Inbound Scores: blocking=5, detection=5, per_pl=5-0-0-0, threshold=5) - (Outbound Scores: blocking=0, detection=0, per_pl=0-0-0-0, threshold=4) - (SQLI=0, XSS=0, RFI=0, LFI=0, RCE=5, PHPI=0, HTTP=0, SESS=0, COMBINED_SCORE=5)
@franbuehler
Copy link
Contributor

The regex of the rule 932260 is assembled from the source with include-except unix-shell-4andup unix-shell-fps-pl1. This means we already have to possiblity to exclude false positives with this file at PL1 unix-shell-fps-pl1.

I'm not entirely sure if the intention is for this file unix-shell-fps-pl1 to be created automatically or whether commands can be added manually later.

@dune73
Copy link
Member

dune73 commented Apr 22, 2024

I'd say we add axel to said file.

@theseion : Do you agree?

@franbuehler
Copy link
Contributor

Thank you! I'll provide a PR, if Max agrees.

@theseion
Copy link
Contributor

I agree. I think it's safe enough to exclude at PL1. The exclusions need to be added to the list manually.

@franbuehler
Copy link
Contributor

Ok, I'll open a PR. Thank you both!

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

Successfully merging a pull request may close this issue.

4 participants