-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
SNS alert ERROR #674
Comments
i think you should add |
Thanks for your reply.
If I keep the default
|
I am having this exact same issue. Any update with a fix? |
The issue here is conflict in the use of "aws_region" config key. It's used as an identifier for the aws authentication code to try and sign requests, https://github.com/Yelp/elastalert/blob/master/elastalert/util.py#L272 And for the SNS alerter, https://github.com/Yelp/elastalert/blob/master/elastalert/alerts.py#L791 This should be fixed, but as an immediate solution, you can nest aws_region INSIDE the alert config so that it will not be used for signing AWS requests. Do this:
Not:
|
Would it be sufficient to rename the key for the sns alerter ? Something like sns_aws_region ? If so I can make a pull request with this modification. |
Fix Document Running Docker container
Hi Quentin,
Thanks for the help for SNS alert setup, now I can pass the test part. But there is still an issue when I start the alerting.
It happens in elastalert trying to sent alert after finding match
The setting of my alert in rule is the following:
Thanks for the help!
The text was updated successfully, but these errors were encountered: