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

Scope requests to view and edit vote record to the specified election #90

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

emalm
Copy link
Contributor

@emalm emalm commented Jun 3, 2023

The controllers to view and to edit a voter's submission in an election now search for that vote record within the scope of the ID of the election.

Otherwise, the controller will find the first vote that user submitted for any election in the system and attempt to decrypt that vote using the supplied password.

This issue arose as the Cloud Foundry community started conducting this year's Technical Oversight Committee election with our Elekto deployment that still had records from the 2022 election. I cast my own vote only to find that the only vote record I could retrieve was my vote from that previous election, with the password I had set for it at the time of submission.

…ction

The controllers to view and to edit a voter's submission in an election now
search for that vote record within the scope of the ID of the election.

Otherwise, the controller will find the first vote that user submitted
for any election in the system and attempt to decrypt that vote using
the supplied password.
@emalm
Copy link
Contributor Author

emalm commented Jun 12, 2023

@jberkus any chance you could review this PR at some point? Thanks!

@jberkus
Copy link
Member

jberkus commented Jun 12, 2023

Thanks. Code looks fine, lemme get it on my test cluster so that I can try it out.

@jberkus
Copy link
Member

jberkus commented Jun 20, 2023

Tested finally. We really need automated tests for "view ballot", sorry.

Merging, thanks so much for the patch. New release coming shortly.

@jberkus jberkus merged commit 131633d into elekto-io:main Jun 20, 2023
@emalm emalm deleted the scope-vote-view-edit-to-election branch June 20, 2023 22:14
@emalm
Copy link
Contributor Author

emalm commented Jun 20, 2023

Thanks again, @jberkus!

@jberkus
Copy link
Member

jberkus commented Jun 20, 2023

Thank you! And if you know anyone who would like to be a co-maintainer on Elekto, send them my way.

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.

2 participants