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

Added rules dir and minor update to config #3098

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

Vetpeet
Copy link

@Vetpeet Vetpeet commented Jan 18, 2021

Added rules dir with 3 x rules (o365_login, legacy_mail_client, blacklist_source_ip)

@nsano-rururu
Copy link
Contributor

in the example_rules directory

@nsano-rururu
Copy link
Contributor

I don't think you need elastalert / alerts.bak. As I pointed out, place the sample in the example_rules directory. Contains fixes that have nothing to do with the title of the pull request. It doesn't make sense.

@Vetpeet
Copy link
Author

Vetpeet commented Apr 6, 2021

@nsano-rururu , apologies we maintain our own fork and I mistakenly pushed to the main repo rather then our fork.

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