solo-io / go-utils

golang utilities
Apache License 2.0
110 stars 18 forks source link

Publish ScanAggregate to slack #466

Closed sam-heilbron closed 2 years ago

sam-heilbron commented 2 years ago

When a security scan completes, publish the ScanAggregate (metadata about the scan) to slack

Context

I had initially attempted to make the scanner more event driven, so that the scanner produced events and we could have a set of subscribers that consumed them. This would make it easier to separate the behavior of how to execute a scan from how to respond to scan events. I had started the work on this branch: https://github.com/solo-io/go-utils/tree/security-scan-notifier. I decided to move away from it, since it was a little too much work for this type of change. If in the future there are more changes to scan eventing, we may want to reconsider using it.