Skip to content
This repository was archived by the owner on Aug 20, 2022. It is now read-only.

Comments

core: Add version to DepositTx#19

Merged
trianglesphere merged 1 commit intooptimism-prototypefrom
jg/versioned_deposits
May 24, 2022
Merged

core: Add version to DepositTx#19
trianglesphere merged 1 commit intooptimism-prototypefrom
jg/versioned_deposits

Conversation

@trianglesphere
Copy link
Contributor

@trianglesphere trianglesphere commented May 17, 2022

Description

This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.

Metadata

  • Fixes ENG-2221

This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
Copy link
Contributor

@protolambda protolambda left a comment

Choose a reason for hiding this comment

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

LGTM

@trianglesphere trianglesphere merged commit 6e3df1b into optimism-prototype May 24, 2022
@trianglesphere trianglesphere deleted the jg/versioned_deposits branch May 24, 2022 00:50
trianglesphere added a commit that referenced this pull request May 24, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Jun 1, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Jun 1, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Jun 2, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Jul 15, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Jul 15, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Aug 2, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
protolambda pushed a commit that referenced this pull request Aug 2, 2022
This adds a second byte after the EIP-2718 Type byte to deposit
transactions that versions the deposit. It is placed prior to the start
of the RLP to make it easier to extend the transaction in the future. We
choose to version inside the EIP-2718 envelope to limit the number of
EIP-2718 prefixes we take for deposit transactions.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants