Skip to content

JSON-RPC: Add eth_createAccessList RPC call - #196

Merged
timbeiko merged 1 commit into
ethereum:masterfrom
MariusVanDerWijden:patch-1
May 26, 2021
Merged

JSON-RPC: Add eth_createAccessList RPC call#196
timbeiko merged 1 commit into
ethereum:masterfrom
MariusVanDerWijden:patch-1

Conversation

@MariusVanDerWijden

Copy link
Copy Markdown
Member

No description provided.

@timbeiko

Copy link
Copy Markdown
Contributor

@lightclient @alita-moore can you have a quick look before I merge? TY!

@timbeiko
timbeiko merged commit 4cad29f into ethereum:master May 26, 2021

@lightclient lightclient left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

1 quick comment.

Comment thread json-rpc/spec.json
"gasUsed": {
"$ref": "#/components/schemas/Integer"
},
"vmErr": {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this field optional? If an error is returned, is accessList and gasUsed omitted?

@lightclient

Copy link
Copy Markdown
Member

oooooof

@MariusVanDerWijden

Copy link
Copy Markdown
Member Author

@timbeiko don't push the button so early 🙈 I wanted to have some discussions on it first

@timbeiko

Copy link
Copy Markdown
Contributor

Oops, sorry! Will revert this.

@timbeiko

Copy link
Copy Markdown
Contributor

@MariusVanDerWijden reverted in faac97e.

Can you open a new PR with the same changes and I'll make sure to leave it open until you tell me to merge :-) ? Thanks!

danceratopz pushed a commit to danceratopz/execution-specs that referenced this pull request Oct 22, 2025
* tools/code: Add len

* tools/vm: add opcode data portion assertion

* vm/opcode: handle strings are hex args

* vm/opcode: add test, expand docstring

* fix: vm error clarity

* types/storage: add auto store counter

* types/storage: add bytes to input args types

* types/storage: add default empty dict to init

* types/storage: storage accepts instances of bytes (Hash32)
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.

4 participants