-
Notifications
You must be signed in to change notification settings - Fork 767
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: eip2930/1559 response type adjustments (part 1) #353
Conversation
use serde::{Deserialize, Serialize}; | ||
|
||
/// Access list | ||
#[derive(Debug, Default, Clone, PartialEq, Serialize, Deserialize, RlpEncodable)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I hit one bug here, RlpEncodable
should be RlpEncodableWrapper
. unless you're using .0
to do the rlp encoding, otherwise you got an extra "[]".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
splitting the transaction mod definitely makes it more maintainable and provides more structure.
pub gateway_fee: Option<U256>, | ||
|
||
// EIP2718 | ||
/// Transaction type, Some(1) for AccessList transaction, None for Legacy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably also mention that Some(2)
is EIP1559
base_fee_per_gas
toBlock
access_list
,transaction_type
,max_priority_fee_per_gas
,max_fee_per_gas
toTransaction
transaction_type
,effective_gas_price
toTransactionReceipt
SignerMiddleware::sign_transaction
now returns an rlp encoded signed transaction instead of aTransaction
Middleware::send_raw_transaction
now receives a signed rlp transaction instead of aTransaction