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

Add two env variables. One for using Rekor public key from OOB and one for fetching it from Rekor server #1610

Merged
merged 1 commit into from
Mar 16, 2022

Conversation

vaikas
Copy link
Contributor

@vaikas vaikas commented Mar 15, 2022

Add two env variables. One for using Rekor public key from OOB and
one for fetching the API from the Rekor API.

Signed-off-by: Ville Aikas vaikas@chainguard.dev

Summary

Add new Environmental variable:
SIGSTORE_REKOR_PUBLIC_KEY

This is kind of like the other SIGSTORE_* that allows you to override some keys / certs. It
allows you to use OOB Rekor Public Key.

Since Rekor API exposes a method for fetching the Public Key it's using, add an env
variable that fetches the public key from the Rekor server.
SIGSTORE_TRUST_REKOR_API_PUBLIC_KEY

This was discovered during the implementation of the Cosigned API work and in particular
validating the keyless images.

I'll be adding e2e tests for this tmw using the sigstore scaffolding.

Ticket Link

Fixes

Release Note

Add two new Environmental variables allowing more control over which Rekor Public Key to use:
SIGSTORE_TRUST_REKOR_API_PUBLIC_KEY - Fetch the Public Key from Rekor
SIGSTORE_REKOR_PUBLIC_KEY - OOB way to specify the Public Key, just like SIGSTORE_ROOT_FILE for example.

one for fetching the API from the Rekor API.

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
@dlorenc dlorenc merged commit f8627f5 into sigstore:main Mar 16, 2022
@github-actions github-actions bot added this to the v1.7.0 milestone Mar 16, 2022
@vaikas vaikas deleted the scaffolding-tests branch March 16, 2022 05:18
vaikas added a commit to vaikas/scaffolding that referenced this pull request Mar 16, 2022
Add `verify-job` job which will run cosign verify.
Bump Fulcio to v0.2.0 release.
Update cosign containers to latest CI builds to pick up
sigstore/cosign#1610

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
vaikas added a commit to sigstore/scaffolding that referenced this pull request Mar 16, 2022
* Change job `check-oidc` name to `sign-job`.
Add `verify-job` job which will run cosign verify.
Bump Fulcio to v0.2.0 release.
Update cosign containers to latest CI builds to pick up
sigstore/cosign#1610

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>

* Forgot to copy and use fulcio public key in verify.

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>

* Add --rekor-url flag to example verify command as well as the env
variable SIGSTORE_TRUST_REKOR_API_PUBLIC_KEY=1

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
coyote240 pushed a commit to coyote240/cosign that referenced this pull request Mar 16, 2022
…igstore#1610)

one for fetching the API from the Rekor API.

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
mlieberman85 pushed a commit to mlieberman85/cosign that referenced this pull request May 6, 2022
…igstore#1610)

one for fetching the API from the Rekor API.

Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
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.

2 participants