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.

Sign up for GitHub

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

Jump to bottom

Migrate to CoSigV1 timestamped signatures #62

Merged
merged 6 commits into from
Dec 7, 2023
Merged

Migrate to CoSigV1 timestamped signatures #62

merged 6 commits into from
Dec 7, 2023

Conversation

AlCutter
Copy link
Contributor

@AlCutter AlCutter commented Dec 6, 2023 •
edited
Loading

This PR:

  • Adds support to the distributor for CoSigV1 signatures (and removes support for vanilla Ed25519 cosigs).
  • Fixes a bug where arbitrary/unknown cosigs on checkpoints would be accepted and integrated into the DB if accompanied by an expected witness signature.

@AlCutter AlCutter requested a review from mhutchinson December 6, 2023 14:16
@AlCutter AlCutter changed the title Timestamped signatures Migrate to CoSigV1 timestamped signatures Dec 6, 2023
@AlCutter AlCutter force-pushed the timestamped_signatures branch from cb6ebd2 to 47554ef Compare December 6, 2023 14:17
@AlCutter AlCutter force-pushed the timestamped_signatures branch from bf91975 to 904f358 Compare December 7, 2023 15:12
@AlCutter AlCutter merged commit f47a82a into transparency-dev:main Dec 7, 2023
3 checks passed
@AlCutter AlCutter deleted the timestamped_signatures branch December 7, 2023 16:20
mhutchinson added a commit to mhutchinson/distributor that referenced this pull request Jan 31, 2024
@mhutchinson
Make tests reflect real signature patterns
23f5df0 
Witnesses are now expected to provide signatures with timestamps embedded since transparency-dev#62.
@mhutchinson mhutchinson mentioned this pull request Jan 31, 2024
Merged
mhutchinson added a commit that referenced this pull request Jan 31, 2024
@mhutchinson
Make tests reflect real signature patterns (#80)
a99f097 
Witnesses are now expected to provide signatures with timestamps embedded since #62.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhutchinson mhutchinson mhutchinson approved these changes

Assignees

@mhutchinson mhutchinson

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AlCutter @mhutchinson