Skip to content

Add RFC3161 timestamps to keyless signer and verifier#979

Merged
aaronlew02 merged 1 commit intomainfrom
timestamp-keyless
Jun 5, 2025
Merged

Add RFC3161 timestamps to keyless signer and verifier#979
aaronlew02 merged 1 commit intomainfrom
timestamp-keyless

Conversation

@aaronlew02
Copy link
Collaborator

Closes #959 and thus closes #699

Summary

This change allows for the KeylessSigner to add RFC3161 timestamps to a bundle and KeylessVerifier to verify such timestamps.

@aaronlew02 aaronlew02 force-pushed the timestamp-keyless branch from b979591 to dd14cf4 Compare June 4, 2025 17:58
@aaronlew02 aaronlew02 force-pushed the timestamp-keyless branch 2 times, most recently from 1e939e9 to 0cbeba6 Compare June 4, 2025 18:40
@aaronlew02 aaronlew02 requested a review from loosebazooka June 4, 2025 18:57
@aaronlew02 aaronlew02 force-pushed the timestamp-keyless branch 2 times, most recently from a4ee8a7 to 579c8d5 Compare June 5, 2025 14:22
@aaronlew02 aaronlew02 requested a review from loosebazooka June 5, 2025 14:30
Copy link
Member

@loosebazooka loosebazooka left a comment

Choose a reason for hiding this comment

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

LGTM, just that minor refactoring.

Signed-off-by: Aaron Lew <64337293+aaronlew02@users.noreply.github.com>
@aaronlew02 aaronlew02 force-pushed the timestamp-keyless branch from 579c8d5 to 3df4c8d Compare June 5, 2025 15:56
@aaronlew02 aaronlew02 requested a review from loosebazooka June 5, 2025 17:37
Copy link
Member

@loosebazooka loosebazooka left a comment

Choose a reason for hiding this comment

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

cool, lets enable the conformance test that used timestamps now that this is in?

@aaronlew02 aaronlew02 merged commit af40879 into main Jun 5, 2025
38 of 39 checks passed
@aaronlew02 aaronlew02 deleted the timestamp-keyless branch June 5, 2025 17:52
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 timestamp to bundle and validate Support validation based on rfc3161 from TSA (vs SET from rekor)

2 participants

Comments