Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

Error: Error fetching UTXO, max retries to connect to DAPI node reached #277

Closed
dashameter opened this issue Jul 10, 2020 · 10 comments
Closed
Labels

Comments

@dashameter
Copy link

Expected Behavior

Current Behavior

Error: Error fetching UTXO yaf8J6Dw7KadZdvYCX3yX1NEc34RUnnCL6,yRvm75mwdxtUmNeg3StuZBboq7U1G8vbSa,ybkPy9jwr9EXyBqynCfZVBjTWD64ZqwPLe,yTV88rRYbSBcPm47Jeq7fqceVFw5NEPi7d,:{_from}:{_to} - max retries to connect to DAPI node reached
at fetchAndReturnUTXO (node_modules/@dashevo/wallet-lib/src/transporters/types/DAPIClientWrapper/methods/getUTXO.js:17:1)
    at async DAPIClientWrapper.getUTXO (node_modules/@dashevo/wallet-lib/src/transporters/types/DAPIClientWrapper/methods/getUTXO.js:21:1)
    at async DAPIClientWrapper.executor (node_modules/@dashevo/wallet-lib/src/transporters/types/DAPIClientWrapper/methods/subscribeToAddressesTransactions.js:29:1)

Context

Nuxt, Webpack

@dashameter dashameter added the bug label Jul 10, 2020
@wizlee
Copy link

wizlee commented Aug 2, 2021

HI @dashameter, is this bug still valid? I am working on an incubator bounty to gather issues and PR fixes from platform projects to create test suites to help platform release stability

@dashameter
Copy link
Author

@wizlee for the testsuite I would assume that this bug is still valid and write a test for it. Maybe coordinate with @Alex-Werner regarding the open bugs since he is also working on a similar overview right now

@wizlee
Copy link

wizlee commented Aug 4, 2021

@dashameter , does it means that this bug is already fix? Didn't see any PR linked to this.

@dashameter
Copy link
Author

@wizlee it's possible that changes in other PRs fixed this bug as well, since the max retries rule has been updated in the server configs. We still want to test against regressions, though.

@wizlee
Copy link

wizlee commented Sep 13, 2021

@dashameter, Alex clarified that this issue is no longer valid because getUTXO is no longer implemented in this repo. As such, can you please help to close this issue?

Paraphrasing @Alex-Werner :

It is now implemented within wallet lib or dash SDK.
An account has a getUTXOS() method that returns them.

A side note, initialisation is also possible from a public address.

For scenarios such as social app where users want the balance of a lot of address, current implementation will make that not very convenient.
Therefore this will be dealt with at some point in the future, ideally by having clearly expressed need from the community to prioritize it (as right now it's still hypothetical).

@Alex-Werner
Copy link
Contributor

Thank you @wizlee !
As such, I think we can close this issue

@SamKirby22
Copy link

@Alex-Werner would you mind reopening this one please. We're tracking it and have a task for retesting and closing. Thanks

@dash-maverick
Copy link

Reopening for additional retest

@dash-maverick dash-maverick reopened this Oct 10, 2021
@dashameter
Copy link
Author

passed retest

@SamKirby22
Copy link

Triage sheet updated. Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

5 participants