You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
// Create the correct signer and signature transform based on the chain ID
varsigner types.Signer
ifchainID==nil {
signer=new(types.HomesteadSigner)
} else {
signer=types.NewEIP155Signer(chainID)
signature[64] -=byte(chainID.Uint64()*2+35)
}
signed, err:=tx.WithSignature(signer, signature)
What are the use-cases?
Can use EIP1559 dynamic fee tx
Implementation
Do you have ideas regarding the implementation of this feature?
No - I have been reading the code and trying to see what all changes I will have to make, but I can give it a shot.
From what I understand this block needs to support typed transactions as ethereum app on ledger supports parsing that.
Rationale
Why should this feature exist?
Currently Ledger only support LegacyTx and I want it to support DynamicTx as the LegacyTx is well, Legacy.
go-ethereum/accounts/usbwallet/ledger.go
Lines 379 to 387 in f59d013
What are the use-cases?
Can use EIP1559 dynamic fee tx
Implementation
Do you have ideas regarding the implementation of this feature?
No - I have been reading the code and trying to see what all changes I will have to make, but I can give it a shot.
From what I understand this block needs to support typed transactions as ethereum app on ledger supports parsing that.
go-ethereum/accounts/usbwallet/ledger.go
Lines 336 to 343 in f59d013
I still don't know how to deal with this though
go-ethereum/accounts/usbwallet/ledger.go
Lines 353 to 357 in f59d013
Are you willing to implement this feature?
Yes
The text was updated successfully, but these errors were encountered: