Skip to content

chore: Implement ErrorReproducibility for crypto errors occurring during IDKG payload validation #5964

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

eichhorl
Copy link
Contributor

@eichhorl eichhorl commented Jul 17, 2025

During IDKG payload validation, non-reproducible errors should be retried. To allow this, we implement the ErrorReproducibility trait for all possible crypto errors.

The validator then determines if the received error should be wrapped into an InvalidIDkgPayloadReason (meaning the payload is reproducibly invalid) or IDkgPayloadValidationFailure (meaning the validation should be retried later) based on the result returned by the ErrorReproducibility implementation.

@eichhorl eichhorl changed the title Draft: impl error reproducibility chore: Implement ErrorReproducibility for crypto errors occurring during IDKG payload validation Jul 17, 2025
@github-actions github-actions bot added the chore label Jul 17, 2025
@eichhorl eichhorl marked this pull request as ready for review July 17, 2025 13:34
@eichhorl eichhorl requested review from a team as code owners July 17, 2025 13:34
Copy link
Contributor

@berestovskyy berestovskyy left a comment

Choose a reason for hiding this comment

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

Interface changes look good, thanks!

Copy link
Contributor

@kpop-dfinity kpop-dfinity left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants