-
Notifications
You must be signed in to change notification settings - Fork 23
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 the 2 new supported v2/verify
payloads
#131
Conversation
✅ Deploy Preview for silly-bunny-8cedd0 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Marking as draft until the dependencies have landed. |
a421edb
to
0a4b4f4
Compare
Updated to match changes in kubewarden/policy-evaluator#152. |
v1/verify
payloadsv2/verify
payloads
@viccuad I think the JSON objects you added are missing a special field used by serde to keep track of the type of enum that is being represented... 🤔 |
@flavio which fields are missing? JSON objects looks good to me. I've used these objects like those for testing the verify-image-signatures policy |
0a4b4f4
to
d606b80
Compare
this is ready for another review, with the latest changes in the callback payloads. |
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
Description
Fix kubewarden/policy-sdk-rust#40
Fix kubewarden/policy-sdk-rust#41
Test
Locally. See also on the automated build.