-
Notifications
You must be signed in to change notification settings - Fork 463
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
[Rule tuning] Fix evasion for disable iptables rule #5
Conversation
💚 CLA has been signed |
FYI I've signed the CLA just before opening the PR.. maybe the CLA bot is not updated yet. 😃 |
I'm not 100% sure how the CLA bot works -- it already knows my GitHub account as an Elastic employee, so I'm kinda exempt. But doesn't hurt to double check the email/github user name you associated with it. And you can check your profile settings to make sure the account is linked: https://github.com/settings/emails |
@rw-access yep, I've inserted the correct GitHub username and email. EDIT: now it's stating that it's signed. 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition @phra, this LGTM - can you also bump the updated_date
Great! I'm less familiar with these commands, so I added a few reviewers that are more familiar with the rule and the particular linux commands to double check the logic |
updated updated_date field
@brokensound77 done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dived into the docs a quick bit, LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
🥂 |
will fix #4
yes
yes