-
Notifications
You must be signed in to change notification settings - Fork 731
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
Add support for pending
in getTransactionCount
#3415
Conversation
@@ -784,7 +786,16 @@ export class Eth { | |||
if (account === undefined) { | |||
return '0x0' |
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.
Was about to say that this should include pending txpool msgs from this account, but this is impossible since accounts do need to exist because otherwise they cannot pay the min basefee of 7 wei.
const rpc = getRpcClient(startRPC(manager.getMethods())) | ||
|
||
const res = await rpc.request(method, ['0xccfd725760a68823ff1e062f4cc97e1360e8d997', 'pending']) |
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.
Not sure, maybe we want to keep this test in it but now instead of using pending
we use some invalid tag like unsupported
or something.
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.
Meh I guess this is tested in the getBlock
anyways
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
This updates the response handler for
eth_getTransactionCount
to support thepending
block option. This includes pending transactions currently in the local transaction pool in the transaction count.This is needed to allow our client to work with the tx fuzzer