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
GQL API for getStorageAt and getLogs #77
Conversation
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.
LGTM. Only issue is in regards to handling the decoding of 2718 typed transactions, which could perhaps be handled as part of #78 instead of here.
for index, receiptBytes := range receiptsBytes { | ||
receiptCID := receiptCIDs[index] | ||
receipt := new(types.Receipt) | ||
if err := rlp.DecodeBytes(receiptBytes, receipt); err != nil { |
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.
For compatibility with EIP-2718 enveloped transactions and receipts (EIP-2930 access list and EIP-1559 dynamic fee) we can no longer use RLP encoding/decoding, we instead need to use the MarshalBinary
and UnmarshalBinary
methods for receipts and transactions.
Related issue and more info here: #78 (comment).
Part of #68. Supersedes #69