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

RSA Validation: Cherry-Picked ont 1.3.2 release #764

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

artoonie
Copy link
Collaborator

@artoonie artoonie commented Oct 30, 2023

Cherry-picking #755 onto the 1.3.2 release

@artoonie artoonie changed the title Feature/rsa validation onto 1.3.2 RSA Validation: Cherry-Picked ont 1.3.2 release Oct 30, 2023
@HEdingfield
Copy link
Contributor

Will leave this to @tarheel to approve, since he approved #755 (also because I'm unsure of how to actually test that this is WAI on my end).

@HEdingfield
Copy link
Contributor

Also seems like we'd want to merge #765 first...

@artoonie
Copy link
Collaborator Author

artoonie commented Nov 5, 2023

We can merge either first. If this one goes in first, I'll cherry-pick the relevant commits in #765 so there are no merge conflicts.

To check if this works:

  1. Run the tests
  2. Download the verity files with .sig.xml hashes (available on Slack) and create a config that loads them. Then hit "Auto-Load Candidates." If the signature is valid, it will work. Change one of the signatures or data files. It will then fail.

@HEdingfield
Copy link
Contributor

To check if this works:

  1. Run the tests
  2. Download the verity files with .sig.xml hashes (available on Slack) and create a config that loads them. Then hit "Auto-Load Candidates." If the signature is valid, it will work. Change one of the signatures or data files. It will then fail.

Trying out 2... there is no auto-load candidates option. Wasn't this something we added in 1.4.0?

@yezr
Copy link
Collaborator

yezr commented Nov 7, 2023

Wasn't this something we added in 1.4.0?

yep! You should also be able to run a tabulation to verify that the signature verification works. It won't tabulate if it can't verify the signature.

Here are some of the illustrious candidates from that test data that you can put in manually

  • Mickey Mouse
  • Bugs Bunny
  • Homer Simpson
  • Woody Woodpecker
  • Scrooge McDuck
  • Toucan Sam

@artoonie
Copy link
Collaborator Author

artoonie commented Nov 8, 2023

Here are some of the illustrious candidates from that test data that you can put in manually

Yeah, sorry, this was the testing process before the cherry-pick.

@artoonie artoonie merged commit 3f96422 into hotfix/1.3.2 Nov 8, 2023
1 check passed
@artoonie artoonie deleted the feature/rsa-validation-onto-1.3.2 branch November 8, 2023 02:39
@HEdingfield
Copy link
Contributor

Tested this out locally with some of the candidates and it failed with "no cast vote records found". Tried it again after modifying one of the sig files and it failed with "Failure while trying to verify hash", so I guess that worked? Ah well, merged now anyway!

@HEdingfield HEdingfield mentioned this pull request Nov 8, 2023
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