Skip to content

Create utility for generating Tink keysets#1627

Merged
Hayden-IO merged 4 commits into
sigstore:mainfrom
Hayden-IO:create-tink-keyset
Jul 30, 2025
Merged

Create utility for generating Tink keysets#1627
Hayden-IO merged 4 commits into
sigstore:mainfrom
Hayden-IO:create-tink-keyset

Conversation

@Hayden-IO

Copy link
Copy Markdown
Contributor

tinkey requires a Java runtime environment and doesn't support outputting PEM-encoded public keys. Threw together this utility to generate keysets and output the keyset's primary key. This will be used when spinning up new log shards to create key material.

Summary

Release Note

Documentation

tinkey requires a Java runtime environment and doesn't support outputting PEM-encoded public keys.
Threw together this utility to generate keysets and output the keyset's primary key.
This will be used when spinning up new log shards to create key material.

Signed-off-by: Hayden B <8418760+haydentherapper@users.noreply.github.com>
@Hayden-IO

Copy link
Copy Markdown
Contributor Author

Ref: sigstore/rekor-tiles#398

Putting this in Scaffolding so it can be used across deployments that need Tink keysets.

Hayden-IO pushed a commit to sigstore/fulcio that referenced this pull request Jul 10, 2025
sigstore/scaffolding#1627 replaces it.

Signed-off-by: Hayden <haydentherapper@users.noreply.github.com>

@jku jku left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems like a good idea.

A mention in the README (or in some docs) would be nice but none of the other tools seem to be documented either so 🤷

Comment thread cmd/create-tink-keyset/app/root.go Outdated
Signed-off-by: Hayden B <8418760+haydentherapper@users.noreply.github.com>
@Hayden-IO

Copy link
Copy Markdown
Contributor Author

Thanks for the README suggestion, I added a doc for the command. We need to update the top-level readme in this repo as well.

bobcallaway
bobcallaway previously approved these changes Jul 17, 2025

@bobcallaway bobcallaway left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

would be nice to get CI to test building this code on PRs this at some point

@Hayden-IO

Copy link
Copy Markdown
Contributor Author

Added CI.

Signed-off-by: Hayden B <8418760+haydentherapper@users.noreply.github.com>
@Hayden-IO Hayden-IO force-pushed the create-tink-keyset branch from 2c018bd to 3a78ea9 Compare July 17, 2025 17:59
Comment thread .github/workflows/create-tink-keyset-test.yml Outdated
@Hayden-IO Hayden-IO requested a review from bobcallaway July 18, 2025 19:24
@Hayden-IO Hayden-IO enabled auto-merge (squash) July 18, 2025 19:25
@Hayden-IO Hayden-IO merged commit bcaa561 into sigstore:main Jul 30, 2025
29 of 31 checks passed
Hayden-IO pushed a commit to sigstore/fulcio that referenced this pull request Jul 30, 2025
sigstore/scaffolding#1627 replaces it.

Signed-off-by: Hayden <haydentherapper@users.noreply.github.com>
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.

4 participants