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

feat: add additional notification policy logic #12

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

RoseSecurity
Copy link

@RoseSecurity RoseSecurity commented Nov 14, 2024

What and Why

Note

Feel free to accept or close this PR. We are still evaluating if this logic has solved our webhook failures


Webhook "datadog-metrics-datadoghq-com" failed with an error: could not deliver webhook payload: Post "https://api.datadoghq.com/api/v2/series": EOF
  • Adds additional logic to the notification policy to assess if the payload is empty. Only sends the webhook if the the payload is non-empty
  • Runs opa fmt on rego policy

References

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.

1 participant