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

Some paths added #5120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CheraghiMilad
Copy link
Contributor

Summary of the Pull Request

Some paths added.

Changelog

Example Log Event

Fixed Issues

SigmaHQ Rule Creation Conventions

  • If your PR adds new rules, please consider following and applying these conventions

@github-actions github-actions bot added Rules Linux Pull request add/update linux related rules labels Dec 10, 2024
Comment on lines +25 to +27
- '/var/lib/docker/containers'
- '/var/lib/mysql'
- '/var/lib/pgsql'
Copy link
Member

Choose a reason for hiding this comment

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

I don't understand the reasoning behind adding these to this rule? These data paths and not log paths

@nasbench nasbench added the Author Input Required changes the require information from original author of the rules label Dec 14, 2024
@nasbench
Copy link
Member

@CheraghiMilad you have to fill the template or I will start closing pull requests :)
Its mandatory to fill it and explain what this PR does.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author Input Required changes the require information from original author of the rules Linux Pull request add/update linux related rules Rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants