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

resolves #516 adds support for private rekor for gitsign attest #517

Merged
merged 4 commits into from
Jun 6, 2024

Conversation

aalsabag
Copy link
Contributor

@aalsabag aalsabag commented Jun 2, 2024

Closes #516

Summary

Gitsign did not support private rekor instances for the gitsign attest command. This adds that feature

Release Note

Fixes gitsign attest's inability to use a private rekor instance

Documentation

NA

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a question and some changes

internal/commands/attest/attest.go Outdated Show resolved Hide resolved
internal/utils_test.go Outdated Show resolved Hide resolved
internal/attest/attest.go Outdated Show resolved Hide resolved
internal/utils.go Outdated Show resolved Hide resolved
internal/attest/attest_test.go Outdated Show resolved Hide resolved
@aalsabag
Copy link
Contributor Author

aalsabag commented Jun 4, 2024

@wlynch these have been addressed. thank you

@wlynch wlynch self-requested a review June 6, 2024 13:54
@wlynch wlynch requested a review from cpanato June 6, 2024 13:54
@cpanato cpanato closed this Jun 6, 2024
@cpanato cpanato reopened this Jun 6, 2024
@cpanato
Copy link
Member

cpanato commented Jun 6, 2024

@aalsabag can you rebase? to trigger the dco

@wlynch
Copy link
Member

wlynch commented Jun 6, 2024

context: DCO broke the other day and might need a new commit to retrigger the flow - dcoapp/app#211

…tsign attest

Signed-off-by: Ahmed Alsabag <ahmed.alsabag@gmail.com>
Signed-off-by: Ahmed Alsabag <ahmed.alsabag@gmail.com>
Signed-off-by: Ahmed Alsabag <ahmed.alsabag@gmail.com>
Signed-off-by: Ahmed Alsabag <ahmed.alsabag@gmail.com>
@aalsabag
Copy link
Contributor Author

aalsabag commented Jun 6, 2024

@wlynch and @cpanato thanks for context. Looks like DCO is passing now.

@cpanato cpanato enabled auto-merge (squash) June 6, 2024 14:35
@cpanato cpanato merged commit bc5ec37 into sigstore:main Jun 6, 2024
7 checks passed
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.

Gitsign attest does not support private rekor instances
3 participants