feat(core): report `Ethereum_EIP7702` from `GetFeatures`
What the developer wrote
Authored by Roman Zeyde
feat(core): report `Ethereum_EIP7702` from `GetFeatures`
1st part of https://github.com/trezor/trezor-firmware/issues/7693.
[no changelog]
What changed, and why it matters
This commit simply advertises that Trezor Model T / Core devices support a new Ethereum feature called EIP-7702 when a host asks the device for its capabilities. It does not change how transactions are signed, authorized, or validated. There is no security-relevant behavior change.
No security action required. Review the actual EIP-7702 implementation separately if assessing security of that feature.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change adds Capability.Ethereum_EIP7702 to the list of capabilities returned by get_features() in core/src/apps/base.py, and updates the corresponding device test to gate on that capability instead of a hard-coded model check. The actual EIP-7702 signing logic is not modified in this commit.
Changed components
core/src/apps/base.pytests/device_tests/ethereum/test_sign_eip7702.pyInspect captured patch +3 / −2
### core/src/apps/base.py
@@ -127,6 +127,7 @@ def get_features() -> Features:
Capability.Cardano,
Capability.Crypto,
Capability.Ethereum,
+ Capability.Ethereum_EIP7702,
Capability.Monero,
Capability.Ripple,
Capability.Stellar,
### tests/device_tests/ethereum/test_sign_eip7702.py
@@ -21,7 +21,7 @@
from trezorlib.debuglink import DebugSession as Session
from trezorlib.ethereum import decode_hex
from trezorlib.exceptions import TrezorFailure
-from trezorlib.messages import EthereumAuth7702, PaymentRequest
+from trezorlib.messages import Capability, EthereumAuth7702, PaymentRequest
from trezorlib.tools import parse_path
from ...common import parametrize_using_common_fixtures
@@ -30,7 +30,7 @@
pytestmark = [
pytest.mark.altcoin,
pytest.mark.ethereum,
- pytest.mark.models("core", reason="T1 does not support EIP 7702"),
+ pytest.mark.capabilities(Capability.Ethereum_EIP7702),
]
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.