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

[#139] Fix /transactions/addresses endpoint return #140

Merged
merged 4 commits into from
Nov 29, 2021

Conversation

brunodb3
Copy link
Contributor

  • Fixes the issue with /transactions/addresses endpoint
    • The indexing was not behaving properly due to an error implementing AwaitLock
    • AwaitLock was getting stuck when multiple listeners were activated, so the data was never saved to the database
  • Fixes/moves leveldb connection handling of errors on .get method
    • Instead of throwing an error when value is not found, returns an empty list. This was done on the leveldb.service previously, but a better place is on the leveldb.connection, as it covers more use-cases
  • Some identation and linting fixes

closes #139

@brunodb3 brunodb3 marked this pull request as ready for review November 25, 2021 00:18
@brunodb3
Copy link
Contributor Author

Tested with redis, the error only happened with leveldb. PR is good to go.

@brunodb3 brunodb3 merged commit 358e0af into master Nov 29, 2021
@brunodb3 brunodb3 deleted the fix-anchor-indexing branch November 29, 2021 09:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Endpoint /transactions/addresses is not returning existing transactions
2 participants