-
Notifications
You must be signed in to change notification settings - Fork 54
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
Support for inclusion/consistency proofs and checkpoint/STH verification. #283
Comments
Checkpoint format comes from https://github.com/sigstore/rekor/blob/main/pkg/util/checkpoint.go. It's Go's SumbDB note format (https://pkg.go.dev/golang.org/x/mod/sumdb/note) with a timestamp in OtherContent. |
Relates to: sigstore#283 Signed-off-by: Victor Embacher <victor@embacher.xyz>
Thanks! I was not sure if there was another specification I did not find. I used the Go code as the reference implementation. |
…added some functionality. Relates to: sigstore#283 Signed-off-by: Victor Embacher <victor@embacher.xyz>
I'm also interested in supporting inclusion proofs, do you need help to get this over the line? |
cc @woodruffw @jleightcap - I think y'all have begun looking at supporting inclusion proof? |
Speaking for the @trailofbits delegation: I don't think we've gotten around to integrating inclusion proofs yet. I was planning on taking a look at the implementation in #285 once the dust settles on bundles (#326 and #311). |
I tried the implementation in #285 today and there's a problem with it, see my comment. Happy to add a commit if you agree with the fix. |
Relates to: sigstore#283 Signed-off-by: Victor Embacher <victor@embacher.xyz>
…added some functionality. Relates to: sigstore#283 Signed-off-by: Victor Embacher <victor@embacher.xyz>
Summary
I would like to see support for inclusion/consistency proofs and checkpoints/STH.
Also related: #274
Reasoning
Implementation
I'm willing to implement this feature, I have already implemented it in another (non-public) crate so I can just port it to this crate.
However, I want to discuss some minor details on how to do it first:
crypto
module that is not be part of the public API.The text was updated successfully, but these errors were encountered: