-
Notifications
You must be signed in to change notification settings - Fork 132
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
Add generic webhook receiver for HMAC signing #127
Conversation
Signed-off-by: Somtochi Onyekwere <[email protected]>
f52d19a
to
f9fc729
Compare
docs/spec/v1beta1/receiver.md
Outdated
HarborReceiver string = "harbor" | ||
DockerHubReceiver string = "dockerhub" | ||
QuayReceiver string = "quay" | ||
GCRReceiver string = "gcr" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs some spacing
internal/server/receiver_handlers.go
Outdated
} | ||
|
||
s.logger.Info( | ||
"handling event from generic wehbook", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"handling event from generic wehbook", | |
"handling event from generic-hmac wehbook", |
Signed-off-by: Somtochi Onyekwere <[email protected]>
Signed-off-by: Somtochi Onyekwere <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #99