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

Change return types to Result in signature verification #30

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

dongsupark
Copy link
Member

@dongsupark dongsupark commented Nov 23, 2023

To deal with signature verification errors in a detailed way, let's change return type of parse_signature_data functions back to Result.

Iterate over signature slots to find a valid signature. If not found, return Error.

See also #28

@dongsupark dongsupark requested review from wrl and pothos November 23, 2023 17:32
@dongsupark dongsupark force-pushed the dongsu/sigverify-return-types branch 2 times, most recently from 3c89abf to 97d19dc Compare November 24, 2023 09:51
To deal with signature verification errors in a detailed way, let's
change return type of parse_signature_data functions back to Result.

Make it iterate over all signature slots to find the first valid
signature. If nothing is found, return error.
@dongsupark dongsupark force-pushed the dongsu/sigverify-return-types branch from 97d19dc to 4e4740c Compare November 24, 2023 16:30
@dongsupark dongsupark merged commit 7b9264d into trunk Nov 27, 2023
1 check passed
@dongsupark dongsupark deleted the dongsu/sigverify-return-types branch November 27, 2023 12:12
dongsupark added a commit to flatcar/scripts that referenced this pull request Dec 7, 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.

2 participants