Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Endpoint /transactions/addresses is not returning existing transactions #139

Closed
brunodb3 opened this issue Nov 23, 2021 · 1 comment · Fixed by #140
Closed

Endpoint /transactions/addresses is not returning existing transactions #139

brunodb3 opened this issue Nov 23, 2021 · 1 comment · Fixed by #140
Labels
bug Something isn't working

Comments

@brunodb3
Copy link
Contributor

brunodb3 commented Nov 23, 2021

On testnet (could be happening on mainnet too), the /transactions/addresses endpoint is not returning what it should, leading to wrong behavior in the web wallet.

The transaction exists on the chain, but the web wallet doesn't display it because of the wrong return of the endpoint.

This potentially has something to do with environment variables. TRANSACTION_INDEXING defaulting to false could mean that /transactions/addresses won't work properly, as it relies on the nodeService.countTransactionsByAddress() method.

If true, the docker-compose on the node repositories should be updated to include TRANSACTION_INDEXING=true.

@brunodb3 brunodb3 added the bug Something isn't working label Nov 23, 2021
@brunodb3
Copy link
Contributor Author

After some investigation, I found that the issue is not related to environment variables, but with some synchronization issue with the listener services. If both anchor-listener and transaction-listener are turned on for example, the anchor-listener kicks in before transaction-listener finishes it's execution, preventing the data to be saved on database.

Further investigation is needed to figure out why this is happening and how to prevent it. But with this, we know for sure that this is also not working properly on mainnet.

brunodb3 added a commit that referenced this issue Nov 29, 2021
[#139] Fix /transactions/addresses endpoint return
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant