Fix github webhook payload detection logic #728
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The correct value for "package_type" seems to be
"CONTAINER"
instead of"docker"
I'm not sure how the original author came up with the value "docker", perhaps that was deprecated/obsoleted by GitHub sometime ago? The documentation at https://docs.github.com/en/webhooks-and-events/webhooks/webhook-events-and-payloads#registry_package doesn't mention anything about potential values.
This was a quick-fix to make GitHub webhook works again but I think if there's time, we should change the detection code to be a bit more robust w/r to GitHub changes (i.e. checks on a well-documented key instead)