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

feat(derive): Add ecrecover trait + features #90

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

clabby
Copy link
Collaborator

@clabby clabby commented Apr 5, 2024

Overview

Adds a new trait, SignedRecoverable, which represents the functionality for recovering a public key from a signed type that has access to the message hash.

With the k256 feature enabled, a default implementation of this trait is implemented on TxEnvelope from alloy-consensus. This feature can be disabled to allow for a custom secp256k1 recovery method. This will be used in the program to facilitate an accelerator for signature recovery during verification through the host program.

If the k256 feature is not enabled, and there is no implementation of the SignedRecoverable extension trait on TxEnvelope, the kona-derive crate will fail to compile.

@clabby clabby added K-feature Kind: feature A-derive Area: kona-derive crate labels Apr 5, 2024
@clabby clabby self-assigned this Apr 5, 2024
remove `custom-ecrecover` feature
@clabby clabby force-pushed the cl/ecrecover-handler branch from 1f7392b to 9087a7b Compare April 5, 2024 02:42
Copy link
Collaborator

@refcell refcell left a comment

Choose a reason for hiding this comment

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

🫳🎤 You dropped this 👑

@refcell refcell added this pull request to the merge queue Apr 5, 2024
Merged via the queue into main with commit d52e6cc Apr 5, 2024
13 checks passed
@github-actions github-actions bot mentioned this pull request Apr 5, 2024
@clabby clabby deleted the cl/ecrecover-handler branch April 5, 2024 03:35
This was referenced May 29, 2024
This was referenced Jun 6, 2024
This was referenced Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-derive Area: kona-derive crate K-feature Kind: feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants