Skip to content

Conversation

@0xferrous
Copy link
Contributor

Motivation

There can be multiple 7702 authorizations provided in a single transaction. So, let's support that.

Solution

We allow at max one unsigned/address as authorization since otherwise it'd be signing multiple authorizations for delegating same account which will just make the last one execute and all the previous ones will essentially be noop. So, for multiple authorizations, the expectation is that signed authorizations will be provided as the args.

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@0xferrous 0xferrous marked this pull request as draft November 21, 2025 16:53
@0xferrous 0xferrous force-pushed the push-svwzwlwpqnow branch 2 times, most recently from bfa0c28 to 8b8193c Compare November 21, 2025 19:37
@0xferrous 0xferrous marked this pull request as ready for review November 21, 2025 19:47
There can be multiple 7702 authorizations provided in a single
transaction. We allow at max one unsigned/address as authorization since
otherwise it'd be signing multiple authorizations for delegating same
account which will just make the last one execute and all the previous
ones will essentially be noop. So, for multiple authorizations, the
expectation is that signed authorizations will be provided as the args.
Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

this lgtm and makes sense

pending @grandizzy

Copy link
Collaborator

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

thank you, checks out. Similar with what we did for cheatcodes in #10623 by switching from an option to a vec of signed auth

@grandizzy grandizzy added this pull request to the merge queue Nov 21, 2025
Merged via the queue into foundry-rs:master with commit d9f47a7 Nov 21, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants