Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Refactor EIP-712 signature verification #1397

Merged
merged 40 commits into from
Nov 7, 2022
Merged
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
7611743
[WIP] EIP-712 Signature Refactor
0a1c Oct 19, 2022
aeda1cf
Merge branch 'main' of https://github.com/evmos/ethermint into austin…
0a1c Oct 20, 2022
e4e8650
Debug and add ante tests
0a1c Oct 20, 2022
afc37f6
Add tests for failure cases
0a1c Oct 21, 2022
57fe441
Add changelog entry
0a1c Oct 21, 2022
998987a
Code cleanup
0a1c Oct 24, 2022
578185c
Add tests for MsgDelegate and MsgWithdrawDelegationReward
0a1c Oct 25, 2022
e941c8e
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
14c4cd7
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
84f061a
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
e93c95e
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
e711f77
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
e97298f
Update ethereum/eip712/encoding.go
0a1c Oct 25, 2022
8f6f63b
Code cleanup
0a1c Oct 25, 2022
876876b
Update ethereum/eip712/encoding.go
0a1c Oct 26, 2022
6b0481f
Minor codefix
0a1c Oct 26, 2022
b8cb5a4
Merge branch 'main' into austin/ENG-891
0a1c Oct 26, 2022
e08c6af
Update ethereum/eip712/encoding.go
fedekunze Oct 26, 2022
74548b7
Minor code revision updates
0a1c Oct 28, 2022
36ea28f
Refactor EIP712 unit tests to use test suite
0a1c Oct 28, 2022
6a7cd65
Merge branch 'main' into austin/ENG-891
facs95 Oct 31, 2022
d91ea3a
update branch
facs95 Oct 31, 2022
13e91de
Address import cycle and implement minor refactors
0a1c Oct 31, 2022
386a019
Fix lint issues
0a1c Oct 31, 2022
3ef98c8
Add EIP712 unit suite test function
0a1c Oct 31, 2022
5003d85
Update ethereum/eip712/encoding.go
0a1c Nov 1, 2022
13a0f4d
Update ethereum/eip712/encoding.go
0a1c Nov 1, 2022
cdba1ee
Update ethereum/eip712/encoding.go
0a1c Nov 1, 2022
f16265a
Add minor refactors; increase test coverage
0a1c Nov 1, 2022
3a5a525
Correct ante_test for change in payload
0a1c Nov 1, 2022
e93b3db
Add single-signer util and tests
0a1c Nov 1, 2022
47e224c
Update ethereum/eip712/encoding.go
fedekunze Nov 2, 2022
1c075c8
Update ethereum/eip712/encoding.go
fedekunze Nov 2, 2022
f795487
Merge branch 'main' into austin/ENG-891
fedekunze Nov 2, 2022
2b7486e
fix build
facs95 Nov 2, 2022
e4795a2
Merge branch 'main' into austin/ENG-891
facs95 Nov 2, 2022
c265a66
Merge branch 'main' into austin/ENG-891
facs95 Nov 3, 2022
a61c40f
Remove reflect import
0a1c Nov 3, 2022
658ed26
Merge branch 'austin/ENG-891' of https://github.com/evmos/ethermint i…
0a1c Nov 3, 2022
be34c8c
Merge branch 'main' into austin/ENG-891
fedekunze Nov 7, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update ethereum/eip712/encoding.go
Co-authored-by: Federico Kunze Küllmer <[email protected]>
  • Loading branch information
0a1c and fedekunze authored Oct 25, 2022
commit 14c4cd71c0f7d4c33bf874ec888fb884babdfc30
3 changes: 1 addition & 2 deletions ethereum/eip712/encoding.go
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,7 @@ func decodeAminoSignDoc(signDocBytes []byte) (apitypes.TypedData, error) {

// Unwrap fees
var fees legacytx.StdFee
err = ethermintAminoCodec.UnmarshalJSON(aminoDoc.Fee, &fees)
if err != nil {
if err := ethermintAminoCodec.UnmarshalJSON(aminoDoc.Fee, &fees); err != nil {
return apitypes.TypedData{}, err
}

Expand Down