Skip to content
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

change default Rekor entry type for intoto attestations to 'dsse' #552

Closed
wants to merge 3 commits into from

Conversation

bobcallaway
Copy link
Member

Fixes: #526

Summary

This causes uploads to Rekor for intoto attestations to use the newly-added dsse type instead of the intoto:0.0.2 type. In addition to being simpler to deal with, Rekor will not persist the attestations for dsse entries.

Release Note

Uploads to Rekor for intoto attestations now use the dsse Rekor type, which will result in the attestation not being stored by Rekor.

Signed-off-by: Bob Callaway <bcallaway@google.com>
Signed-off-by: Bob Callaway <bcallaway@google.com>
@bobcallaway bobcallaway requested a review from a team as a code owner June 12, 2023 23:49
@changeset-bot
Copy link

changeset-bot bot commented Jun 12, 2023

🦋 Changeset detected

Latest commit: 50b8877

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
sigstore Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@bdehamer
Copy link
Collaborator

@bobcallaway thanks for all the work on the support for the new Rekor type. For the npm use case we still have some work to do on the registry to support the dsse type so we won't be merging this until we've got everything ready on the back-end.

@bdehamer
Copy link
Collaborator

With the recent refactoring of the signing logic, this PR is no longer valid. I'm going to close it in favor of #631 which does the same thing (changes the default Rekor type from "intoto" to "dsse")

@bdehamer bdehamer closed this Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

change default rekor type for attestations from 'intoto:0.0.2' to 'dsse'
2 participants