Skip to content
This repository has been archived by the owner on Mar 24, 2023. It is now read-only.

eth_getFilterChanges query filterId from eth_newPendingTransactionFilter inconsistent with other chains #272

Open
gpBlockchain opened this issue Apr 11, 2022 · 3 comments
Labels
godwoken P-Low question Further information is requested

Comments

@gpBlockchain
Copy link

env:https://godwoken-testnet-web3-v1-rpc.ckbapp.dev/

code

let  filterId = await ethers.provider.send("eth_newPendingTransactionFilter",[]);
await sendTxCount(ethers.provider,10)
let logs = await ethers.provider.send("eth_getFilterChanges", [filterId]);
console.log(logs);

result

godwoken

send successful: 10
[]

hardhat

send successful: 10
[
  '0x1aac02dc1132dccfc3d685d34e202e3d2a28bb8b7f5234f7309570365d948e9a',
  '0x3c0636d6123fdd93fe5a8ea69d957438daf23f7bed0d33896dead31b63d18523',
  '0x13fa0ea1c7fa0e688c9108f52b31cbd2c172bcd16a07e79eb5aa2a5490529c9e',
  '0xfb49e663281b73bd08a1c5b9ccc3bb2c3f6e330d529cb40cfcfa0fc0a38e995c',
  '0x43a46279323e1ac835ab1f0f2f35cc6d10b195da1d8dac7f76ccdf27075add4f',
  '0x82f25d175826c5306bc53cc2bf6cc2bd102d1e011862cff1a69f9c0ef5f0c5a6',
  '0xc1cf5dbd1d6e5df6341cf0ff9ff54300de8eeb4d38d138e624c774711c795fdf',
  '0xe1b1c580a618adf8a8150d704eebc3e9a6e3312200f244b55bf711ee2fde48b4',
  '0x1dfea99dbdf040b0f182f856ca8f23b2df5a0aeb17139e8a113d14b7c869044f',
  '0xeee0fa0653596bb24ddaa205e4d057418a8f0c99e719d1215adeab02edfb73b6'
]
@RetricSu
Copy link
Contributor

currently web3 not supporting pending transaction(cc @classicalliu can you confirm this?). so it always return empty [].

@classicalliu
Copy link
Contributor

currently web3 not supporting pending transaction(cc @classicalliu can you confirm this?). so it always return empty [].

You mean notify new pending transactions ? This is really not supported now.

@RetricSu
Copy link
Contributor

currently web3 not supporting pending transaction(cc @classicalliu can you confirm this?). so it always return empty [].

You mean notify new pending transactions ? This is really not supported now.

yes. thanks for clarifying.

@RetricSu RetricSu added question Further information is requested godwoken P-Low labels Apr 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
godwoken P-Low question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants