Skip to content

Commit

Permalink
Merge pull request #1 from livingrockrises/feat/erc7579-proposed-changes
Browse files Browse the repository at this point in the history
updates regarding fallback
  • Loading branch information
kopy-kat authored Aug 13, 2024
2 parents 102cf21 + 9b79f3e commit 9d34a08
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions ERCS/erc-7579.md
Original file line number Diff line number Diff line change
Expand Up @@ -218,10 +218,10 @@ Smart accounts MAY implement a fallback function that forwards the call to a fal

If the smart account has a fallback handler installed, it:

- MUST implement authorization control
- MUST use `call` to invoke the fallback handler
- MUST use `call` or `staticcall` to invoke the fallback handler
- MUST utilize [ERC-2771](./eip-2771.md) to add the original `msg.sender` to the `calldata` sent to the fallback handler
- MUST route to fallback handlers based on the function selector of the calldata
- MAY implement authorization control, which SHOULD be done via hooks

#### ERC-165

Expand Down Expand Up @@ -305,7 +305,7 @@ Executors MUST implement the `IModule` interface and have module type id: `2`.

Fallback handlers MUST implement the `IModule` interface and have module type id: `3`.

Fallback handlers that implement authorization control, MUST NOT rely on `msg.sender` for authorization control but MUST use ERC-2771 `_msgSender()` instead.
Fallback handlers MAY implement authorization control. Fallback handlers that do implement authorization control, MUST NOT rely on `msg.sender` for authorization control but MUST use ERC-2771 `_msgSender()` instead.

#### Hooks

Expand Down

0 comments on commit 9d34a08

Please sign in to comment.