Skip to content
This repository has been archived by the owner on Nov 23, 2023. It is now read-only.

Add support for Ethereum EIP 1559 #871

Closed
wants to merge 21 commits into from
Closed

Add support for Ethereum EIP 1559 #871

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
25b1a85
Update Trezor Common
FrederikBolding Aug 10, 2021
5bd7919
Initial JS implementation
FrederikBolding Aug 10, 2021
d26c037
Fix wrong arguments
FrederikBolding Aug 10, 2021
46042d2
Simplify implementation to use one method
FrederikBolding Aug 10, 2021
1473879
Simplify further
FrederikBolding Aug 10, 2021
5b9792f
Dont do chain id calculation for EIP 1559
FrederikBolding Aug 11, 2021
63bdd2e
Add unavailableCapabilities
FrederikBolding Aug 12, 2021
cdc3fd0
Update test case
FrederikBolding Aug 12, 2021
c8db945
Add access lists
FrederikBolding Aug 12, 2021
c0a8d2a
Small access list fixes
FrederikBolding Aug 12, 2021
b1f322d
Fix access_list fixture
FrederikBolding Aug 12, 2021
3cd99a2
Add more tests
FrederikBolding Aug 12, 2021
fbd990a
Add TS types
FrederikBolding Aug 12, 2021
567c7c8
Fix stripping on access list
FrederikBolding Aug 12, 2021
7b52afc
Update fixtures
FrederikBolding Aug 12, 2021
a0e3def
Update fixture
FrederikBolding Aug 12, 2021
53617b2
Run protobuf gen again
FrederikBolding Aug 12, 2021
71ec905
Fix a few small mistakes in fixtures
FrederikBolding Aug 12, 2021
a4690af
Fix linting
FrederikBolding Aug 12, 2021
f2651a0
Update fixture
FrederikBolding Aug 12, 2021
51340f3
Update fixture
FrederikBolding Aug 12, 2021
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
7 changes: 6 additions & 1 deletion src/data/config.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,11 @@
"excludedMethods": ["decreaseOutput"],
"min": ["1.10.0", "2.4.0"],
"comment": ["allow reduce output in RBF transaction since /2.4.0"]
}
},
{
"excludedMethods": ["eip1559"],
"min": ["0", "2.4.2"],
"comment": ["new eth transaction pricing mechanism (EIP1559) since 2.4.2"]
}
]
}
347 changes: 229 additions & 118 deletions src/data/messages/messages.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2957,13 +2957,6 @@
"type": "uint32",
"name": "pages",
"id": 2
},
{
"rule": "optional",
"options": {},
"type": "uint32",
"name": "page_number",
"id": 3
}
],
"enums": [
Expand Down Expand Up @@ -5103,6 +5096,120 @@
"options": {},
"oneofs": {}
},
{
"name": "EthereumSignTxEIP1559",
"fields": [
{
"rule": "repeated",
"options": {},
"type": "uint32",
"name": "address_n",
"id": 1
},
{
"rule": "required",
"options": {},
"type": "bytes",
"name": "nonce",
"id": 2
},
{
"rule": "required",
"options": {},
"type": "bytes",
"name": "max_gas_fee",
"id": 3
},
{
"rule": "required",
"options": {},
"type": "bytes",
"name": "max_priority_fee",
"id": 4
},
{
"rule": "required",
"options": {},
"type": "bytes",
"name": "gas_limit",
"id": 5
},
{
"rule": "optional",
"options": {
"default": ""
},
"type": "string",
"name": "to",
"id": 6
},
{
"rule": "required",
"options": {},
"type": "bytes",
"name": "value",
"id": 7
},
{
"rule": "optional",
"options": {
"default": ""
},
"type": "bytes",
"name": "data_initial_chunk",
"id": 8
},
{
"rule": "required",
"options": {},
"type": "uint32",
"name": "data_length",
"id": 9
},
{
"rule": "required",
"options": {},
"type": "uint32",
"name": "chain_id",
"id": 10
},
{
"rule": "repeated",
"options": {},
"type": "EthereumAccessList",
"name": "access_list",
"id": 11
}
],
"enums": [],
"messages": [
{
"name": "EthereumAccessList",
"fields": [
{
"rule": "required",
"options": {},
"type": "string",
"name": "address",
"id": 1
},
{
"rule": "repeated",
"options": {},
"type": "bytes",
"name": "storage_keys",
"id": 2
}
],
"enums": [],
"messages": [],
"options": {},
"oneofs": {}
}
],
"options": {},
"oneofs": {}
},
{
"name": "EthereumTxRequest",
"fields": [
Expand Down Expand Up @@ -8912,117 +9019,6 @@
"options": {},
"oneofs": {}
},
{
"ref": "google.protobuf.EnumValueOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_in",
"id": 50002
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_out",
"id": 50003
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_debug_in",
"id": 50004
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_debug_out",
"id": 50005
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_tiny",
"id": 50006
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_bootloader",
"id": 50007
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_no_fsm",
"id": 50008
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "bitcoin_only",
"id": 60000
}
]
},
{
"ref": "google.protobuf.EnumOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "has_bitcoin_only_values",
"id": 51001
}
]
},
{
"ref": "google.protobuf.MessageOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "unstable",
"id": 52001
},
{
"rule": "optional",
"options": {},
"type": "uint32",
"name": "wire_type",
"id": 52002
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "include_in_bitcoin_only",
"id": 60000
}
]
},
{
"ref": "google.protobuf.FieldOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "experimental",
"id": 53001
}
]
},
{
"name": "RippleGetAddress",
"fields": [
Expand Down Expand Up @@ -10599,6 +10595,117 @@
],
"options": {},
"oneofs": {}
},
{
szymonlesisz marked this conversation as resolved.
Show resolved Hide resolved
"ref": "google.protobuf.EnumValueOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_in",
"id": 50002
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_out",
"id": 50003
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_debug_in",
"id": 50004
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_debug_out",
"id": 50005
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_tiny",
"id": 50006
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_bootloader",
"id": 50007
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "wire_no_fsm",
"id": 50008
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "bitcoin_only",
"id": 60000
}
]
},
{
"ref": "google.protobuf.EnumOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "has_bitcoin_only_values",
"id": 51001
}
]
},
{
"ref": "google.protobuf.MessageOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "unstable",
"id": 52001
},
{
"rule": "optional",
"options": {},
"type": "uint32",
"name": "wire_type",
"id": 52002
},
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "include_in_bitcoin_only",
"id": 60000
}
]
},
{
"ref": "google.protobuf.FieldOptions",
"fields": [
{
"rule": "optional",
"options": {},
"type": "bool",
"name": "experimental",
"id": 53001
}
]
}
],
"enums": [
Expand Down Expand Up @@ -11175,6 +11282,10 @@
"name": "MessageType_EthereumSignTx",
"id": 58
},
{
"name": "MessageType_EthereumSignTxEIP1559",
"id": 452
},
{
"name": "MessageType_EthereumTxRequest",
"id": 59
Expand Down
Loading