-
Notifications
You must be signed in to change notification settings - Fork 223
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
Send webhook for deleted reports #1075
Conversation
@bgoareguer thanks you for the contribution , I have added few comments , also maybe it should configurable as this change might break the default integration with postee alerts and postee template , which use rego --> |
@bgoareguer thank you for the PR update , I have added couple of suggestion and |
@bgoareguer look good 🚀 thank you for the contribution, last thing , do you mind updating the helm readme with new param |
lgtm 🚀 |
Description
Before this PR, trivy-operator only sent webhooks for created/updated reports.
In this PR trivy-operator also sends webhooks for deleted reports.
To make sure the receiver of the webhook can distinguish between created/updated reports and deleted reports, the webhook has been modified so that it now contains 2 fields:
Related issues
Checklist