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: support contracts wallet signatures eip1654 #1

Merged
merged 4 commits into from
Feb 4, 2020

Conversation

nachomazzara
Copy link
Contributor

@nachomazzara nachomazzara commented Jan 31, 2020

Support for contract wallet signatures which follows the EIP1654.

Changed

ECDSA_EPHEMERAL_VALIDATOR to ECDSA_PERSONAL_EPHEMERAL_VALIDATOR

Methods

  • getEphemeralSignatureType
  • createEIP1271MessageHash
  • ECDSA_EIP_1654_EPHEMERAL_VALIDATOR

src/types.ts Outdated Show resolved Hide resolved
Co-Authored-By: Esteban Ordano <esteban@decentraland.org>
@@ -14,13 +19,18 @@ export class Authenticator {
/** Validate that the signature belongs to the Ethereum address */
static async validateSignature(
expectedFinalAuthority: string,
authChain: AuthChain
authChain: AuthChain,
provider: any
Copy link

Choose a reason for hiding this comment

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

Should this be an EthereumProvider?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch!

@nachomazzara nachomazzara merged commit a9db006 into master Feb 4, 2020
@nachomazzara nachomazzara deleted the feat/add-ERC-1654 branch February 4, 2020 16:18
@nachomazzara
Copy link
Contributor Author

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants