Skip to content

Conversation

tetsuo-cpp
Copy link
Contributor

Closes #29

@tetsuo-cpp
Copy link
Contributor Author

tetsuo-cpp commented May 10, 2022

I can confirm this works on the staging instance. CSR support hasn't been rolled out to prod yet so we shouldn't merge this right now.

It'll be good to have this ready to go though.

@tetsuo-cpp tetsuo-cpp requested review from di and woodruffw May 10, 2022 12:48
Copy link
Member

@di di left a comment

Choose a reason for hiding this comment

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

LGTM, assuming that the Fulcio-specific serialization is correct.

@woodruffw woodruffw merged commit a000197 into main May 10, 2022
@woodruffw woodruffw deleted the alex/csr-support branch May 10, 2022 17:15
woodruffw added a commit that referenced this pull request May 10, 2022
woodruffw added a commit that referenced this pull request May 10, 2022
javanlacerda pushed a commit to javanlacerda/sigstore-python that referenced this pull request Feb 23, 2024
* treewide: Use OIDC beacon instead of ambient credentials

Signed-off-by: Alex Cameron <[email protected]>

* action: Remove stale TODO

Signed-off-by: Alex Cameron <[email protected]>

* action: Use `ZipFile` context manager

Signed-off-by: Alex Cameron <[email protected]>

* action: Use `rstrip` to strip newline

Signed-off-by: Alex Cameron <[email protected]>

* action: Use `GITHUB_TOKEN` environment variable instead of having a
custom one

Signed-off-by: Alex Cameron <[email protected]>

* README: Add `--identity-token` flag to protocol doc

Signed-off-by: Alex Cameron <[email protected]>

---------

Signed-off-by: Alex Cameron <[email protected]>
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.

Request certificates via CSR
3 participants