-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding draft of Ethereum Signed Transactions #2
base: main
Are you sure you want to change the base?
Conversation
ghost
commented
Nov 9, 2022
- Draft of Ethereum Signed Transactions #1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple notes!
|
||
#### Logic | ||
|
||
This should be implemented in the `crypto` package in the file `secp.go` line 65 as try/catch logic incase the traditional signature verification fails, we can check for this type of signature. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should permalink this as line numbers change.
digestHash := Hasher(ethMsg) | ||
signature := s.Sign(digestHash) | ||
v := new(Secp256k1Validator) | ||
pubk, err := v.Validate(ethMsg, signature) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting
|
||
#### Alternative Solutions | ||
|
||
Creating our own browser plugin wallet is a possibility, but it would be a lot of work and would require a lot of testing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recall seeing metamask having some kind of alternative plugin model or something here, no?