Skip to content

[Draft] Implement EIP-7708#1155

Closed
souradeep-das wants to merge 0 commit intoethereum:masterfrom
souradeep-das:master
Closed

[Draft] Implement EIP-7708#1155
souradeep-das wants to merge 0 commit intoethereum:masterfrom
souradeep-das:master

Conversation

@souradeep-das
Copy link
Contributor

What was wrong?

This is a sample implementation of EIP-7708 (https://eips.ethereum.org/EIPS/eip-7708)

How was it fixed?

As per EIP-7708, the following cases of ETH transfers would now emit a log
a) non-zero value transfer tx (ETH transfer)
b) non-zero value invocation of CALL
c) non-zero value associated with SELFDESTRUCT

Other conditions:

  • Logs use a unique placeholder for MAGIC (topic 1/signature)
  • Logs from value transfer txs would be placed before the other cases of logs (mentioned above)
  • Logs emitted do not account for any additional gas expenses (as per EIP)

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@souradeep-das souradeep-das marked this pull request as draft March 12, 2025 12:56
Carsons-Eels pushed a commit to Carsons-Eels/execution-specs that referenced this pull request Oct 16, 2025
…m contracts address update (ethereum#1155)

* feat(tools): Check and verify tx hash and sender on deploy contract tests

* fix(forks): Update 2935, 7002 and 7251 system contract addresses

* fix(tests): Update 2935, 7002 and 7251 system contract addresses

* changelog

* docs: CHANGELOG

* docs: fix
danceratopz pushed a commit to danceratopz/execution-specs that referenced this pull request Oct 22, 2025
…m contracts address update (ethereum#1155)

* feat(tools): Check and verify tx hash and sender on deploy contract tests

* fix(forks): Update 2935, 7002 and 7251 system contract addresses

* fix(tests): Update 2935, 7002 and 7251 system contract addresses

* changelog

* docs: CHANGELOG

* docs: fix
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