Add authorization status to EIP7702 authorizations in transaction receipt #640
Conversation
Co-authored-by: jxom <[email protected]>
|
Supporting this PR!! Able to determine which authorization succeed helps both developer and wallet user learn about their transaction status. Deducing authorization result through state changes of the transaction are less intuitive. |
There was a problem hiding this comment.
Generally onboard with this. I think it can be computed on-the-fly by clients relatively easily. Felix made a good point on ACD that it requires the authorizer's nonce at the time of the transaction so not possible to compute on the fly without an archive node.
| authorizationStatus: | ||
| title: list of authorizations' status | ||
| description: Ordered list of authorizations' status. List contains either 1 (success) or 0 (failure) for each authorization in the transaction | ||
| type: array | ||
| items: | ||
| $ref: '#/components/schemas/uint' |
There was a problem hiding this comment.
what do you think about returning the list of addresses that were set on behalf of the delegation (null for failure maybe?)
There was a problem hiding this comment.
just to ensure we are on the same page, you are suggesting to return the list of signing accounts in the authorization list, and not the delegation right?
we are agreeable on the former, this would also work for us
Currently, to determine the results of both the transaction and delegation requires multiple RPC calls, which is a suboptimal experience.
We propose a change to the current getTransactionReceipt, to add a list of authorization statuses, so that we are able to determine both results in 1 RPC call.