Full tx subscription missing 'from' field into transaction details

[julesGoullee]

System information

OS & Version: OSX
Commit hash : 34b065a

When subscribing to the pending transactions with the fullTx parameter the field from is missing, which is inconsistent with native geth and also expected from a library like etherjs

@lmittmann

[weiihann]

Can you show the method of reproducing it (i.e. commands)?

[julesGoullee]

It's related to pending transaction filter with full tx: eth_subscribe, this have been introduce here #1626

[joseddg92]

Here is a way to reproduce it:

import logging

from web3 import Web3

IPC_PATH = "/root/test-bsc-node/node/geth.ipc"

w3 = Web3(Web3.IPCProvider(IPC_PATH))
def _pending_tx_loop():
    tx_filter = w3.eth.filter('pending')
    while True:
        try:
            new_pending_txs = tx_filter.get_new_entries()
            for tx in new_pending_txs:
                print(f"{tx}")
        except ValueError:
            logging.exception("_pending_tx_loop")


def main():
    _pending_tx_loop()


if __name__ == "__main__":
    main()

[weiihann]

Hello :) thanks for providing the details. I did some code digging and here's the outcome:

Essentially, #1626 was a cherry-pick PR for this feature, but it wasn't a complete one. Upon investigation, I found that we also need to merge these geth PRs:

1. eth/filters: add global block logs cache ethereum/go-ethereum#25459
2. eth/filters: send rpctransactions in pending-subscription ethereum/go-ethereum#26126
3. eth/filters: fix a breaking change and return rpctransaction ethereum/go-ethereum#26757

One of our team members is working on merging all the past PRs from geth to BSC, and expect it would be live sometime next month. By that time, this feature should be completely resolved.

Hope that helps!

[joseddg92]

Very unfortunate that, due to the fork, updating is mandatory now. And this fiuncionality is broken.