Skip to content

fix: Update restricted files to include Perl subdirectories#4620

Merged
EsadCetiner merged 2 commits into
coreruleset:mainfrom
HackingRepo:patch-4
May 1, 2026
Merged

fix: Update restricted files to include Perl subdirectories#4620
EsadCetiner merged 2 commits into
coreruleset:mainfrom
HackingRepo:patch-4

Conversation

@HackingRepo
Copy link
Copy Markdown
Contributor

@HackingRepo HackingRepo commented Apr 20, 2026

What?

Add perl/ common subdirs, like lib and bin and etc

Why?

Because perl/ alone can cause fps, in the previous pr perl/ caused FPs because of perl/syntaxes, but after that, That still detects probe is a cpan package installed or not while reducing those FPs, what do you think @EsadCetiner

# What?
Add perl/ common subdirs, like lib and bin and etc
# Why?
Because perl/ alone can cause fps
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 20, 2026

📊 Quantitative test results for language: eng, year: 2023, size: 10K, paranoia level: 1:
🚀 Quantitative testing did not detect new false positives

@fzipi
Copy link
Copy Markdown
Member

fzipi commented Apr 20, 2026

Can you link the previous PR here? I'm not 100% convinced yet.

@HackingRepo
Copy link
Copy Markdown
Contributor Author

Hello, @fzipi

PRs associated with that

1- #4607
2- #4536

@fzipi
Copy link
Copy Markdown
Member

fzipi commented Apr 20, 2026

@HackingRepo PLEASE STOP TAGGING ME 😄

@HackingRepo
Copy link
Copy Markdown
Contributor Author

sorry, i will stop for doing that, no issue

Comment thread rules/restricted-files.data Outdated
@HackingRepo HackingRepo requested a review from EsadCetiner May 1, 2026 07:54
@EsadCetiner EsadCetiner added this pull request to the merge queue May 1, 2026
Merged via the queue into coreruleset:main with commit 8710536 May 1, 2026
8 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.

3 participants