jl777 / SuperNET

57 stars 222 forks source link

Electrum balances not showing properly #833

Closed lukechilds closed 6 years ago

lukechilds commented 6 years ago

This looks like it is unrelated to the new perf improvements you pushed to dev, I'm getting this problem in BarterDEX Marketmaker v1.0.164.

I got some Doge sent to my address (thanks @imylomylo) which has definitely confirmed:

https://dogechain.info/address/DLd2e833VJUDJVCpowFFLEcpfzAwTz2sv8

However markermaker doesn't show me having any Doge balance in the response to the portfolio command. It's been about 20 mins since the TX was sent.

I thought it might be a cache issue and triggering listunspent would fix it but that didn't work:

_api.debug({method: 'listunspent', coin: 'DOGE', address: 'DLd2e833VJUDJVCpowFFLEcpfzAwTz2sv8'})
// => []

Also I thought balance would do directly through to the electrum server but that didn't work either:

_api.debug({method: 'balance', coin: 'DOGE', address: 'DLd2e833VJUDJVCpowFFLEcpfzAwTz2sv8'})
// => {result: "success", coin: "DOGE", address: "DLd2e833VJUDJVCpowFFLEcpfzAwTz2sv8", balance: 0}

But if I directly connect to the same Electrum server and query it, I get the correct balance:

$ telnet electrum1.cipig.net 10060
Trying 13.230.86.115...
Connected to electrum1.cipig.net.
Escape character is '^]'.
{ "id": 1, "method":"blockchain.address.get_balance", "params":["DLd2e833VJUDJVCpowFFLEcpfzAwTz2sv8"] }
{"jsonrpc": "2.0", "result": {"confirmed": 30100000000, "unconfirmed": 0}, "id": 1}
jl777 commented 6 years ago

pushed fix

lukechilds commented 6 years ago

Thanks, that's fixed it!

@artemii235 Can we get a new build?

artemii235 commented 6 years ago

@lukechilds Sure, going to merge changes to my repo now.

artemii235 commented 6 years ago

@lukechilds Done: https://github.com/artemii235/SuperNET/releases/tag/v1.0.168

lukechilds commented 6 years ago

@artemii235 Thanks!