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 explicit identity token argument to SigstoreSigner. #319

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

spencerschrock
Copy link
Contributor

Summary

This matches functionality in sigstore-python, allowing callers to provide an identity token provided out-of-band.

https://github.com/sigstore/sigstore-python/blob/cac62e8e611d4286a49ecda7f33d356381ab7919/sigstore/_cli.py#L658-L665

Fixes #316

Release Note

Added an optional identity_token argument to Sigstore signer classes, allowing users to specify which OIDC token to use.

Documentation

This matches functionality in `sigstore-python`, allowing callers to
provide an identity token provided out-of-band.

https://github.com/sigstore/sigstore-python/blob/cac62e8e611d4286a49ecda7f33d356381ab7919/sigstore/_cli.py#L658-L665

Signed-off-by: Spencer Schrock <[email protected]>
@spencerschrock spencerschrock requested review from a team as code owners November 11, 2024 18:34
@spencerschrock
Copy link
Contributor Author

I forgot the CLI part, I'll add it first

Copy link
Collaborator

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

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

Thank you! This looks good and we can do the CLI update in the next PR. Up to you

@spencerschrock
Copy link
Contributor Author

I'll send it as a follow up then

@mihaimaruseac mihaimaruseac merged commit a0fa173 into sigstore:main Nov 11, 2024
21 checks passed
@spencerschrock spencerschrock deleted the oob-token branch November 11, 2024 21:35
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.

Add support for providing identity tokens obtained out-of-band
2 participants