Skip to content

Conversation

@wnxd
Copy link

@wnxd wnxd commented Jan 16, 2026

  • Introduced a new signature scheme SchemeSECP256K1EIP191 for EIP-191 compliant message signing.
  • Implemented a new ecdsaEIP191Key type to handle EIP-191 signing logic.
  • Updated the sign method to generate EIP-191 compliant signatures using the Keccak256 hash function.
  • Modified the key creation logic to accommodate the new signing scheme.

- Introduced a new signature scheme `SchemeSECP256K1EIP191` for EIP-191 compliant message signing.
- Implemented a new `ecdsaEIP191Key` type to handle EIP-191 signing logic.
- Updated the `sign` method to generate EIP-191 compliant signatures using the Keccak256 hash function.
- Modified the key creation logic to accommodate the new signing scheme.
@socket-security
Copy link

socket-security bot commented Jan 16, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgolang.org/​x/​crypto@​v0.47.074100100100100
Addedgithub.com/​stretchr/​testify@​v1.10.096100100100100
Addedgithub.com/​decred/​dcrd/​dcrec/​secp256k1/​v4@​v4.4.0100100100100100

View full report

wnxd added 4 commits January 16, 2026 14:26
…EIP-191 message signing using golang.org/x/crypto/sha3 instead.
…emented EIP-191 message signing using golang.org/x/crypto/sha3 instead."

This reverts commit 288c2ab.
…EIP-191 message signing using golang.org/x/crypto/sha3 instead.
Copy link
Author

@wnxd wnxd left a comment

Choose a reason for hiding this comment

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

Introduced a new signature scheme SchemeSECP256K1EIP191 for EIP-191 compliant message signing.
Implemented a new ecdsaEIP191Key type to handle EIP-191 signing logic.
Updated the sign method to generate EIP-191 compliant signatures using the Keccak256 hash function.
Modified the key creation logic to accommodate the new signing scheme.

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.

1 participant