aliceclaim value not set in values array
#920
Comments
|
This bug doesn't seem to be consistent, here is another trade that also did an {
"uuid": "b66ebae5fb25bebb32a68c724e9dd2913a2a30b4a7304eb26b37a2ce0b1b32dd",
"expiration": 1530105049,
"tradeid": 131541147,
"requestid": 1623453337,
"quoteid": 1904200497,
"iambob": 0,
"Bgui": "",
"Agui": "hyperdex",
"gui": "hyperdex",
"bob": "CHIPS",
"srcamount": 0.10125565,
"bobtxfee": 0.0001,
"alice": "KMD",
"destamount": 0.0134864,
"alicetxfee": 0.00001,
"aliceid": "5319286380928565249",
"result": "success",
"status": "finished",
"finishtime": 1530695014,
"bobdeposit": "09b116ed0123421ba5c4dbc1f44533e381de23e65508c2f5c0b9d7e44fdbc6c8",
"alicepayment": "54feb40c7a513c7eafee9610dd32e89709b329de37739d36b9cd6e089af5d0de",
"bobpayment": "0000000000000000000000000000000000000000000000000000000000000000",
"paymentspent": "0000000000000000000000000000000000000000000000000000000000000000",
"Apaymentspent": "0000000000000000000000000000000000000000000000000000000000000000",
"depositspent": "89732f00f62ad3b4f83fb32410d76ed92312addd5249b273d4aac36c8e403b40",
"method": "tradestatus",
"values": [
0,
0,
0,
0.0135064,
0.1141126,
0,
0.00001735,
0,
0,
0,
0.1140126
],
"sentflags": [
"alicepayment",
"bobdeposit",
"myfee",
"aliceclaim"
]
} |
|
could it be timing based? it could be that it cant get the value, but then gets to a .finished state. after that it wont do any processing. So the way to see if it locked onto a null value due to timing, you can delete .finished and do another swapstatus and see if the aliceclaim value is detected |
|
Doesn't appear to be that, I just tried kickstarting and it still doesn't display the |
|
very strange. you have the txid for the aliceclaim and it is valid and has the right value? |
|
Yeah, linked in the original post ☝️ |
|
I can add some error checking to detect such cases and try to get the value, but if it isnt getting the value now, not sure if retry will work... in this edge case could you approximate by using the bobdeposit value for aliceclaim? |
|
I can for now, that should make the error less noticeable to users. But still, I feel a little uncomfortable about showing users approximations of what should be absolute values. If there's any possible way you can fix it that would be perfect 👌. |
|
I can try to reproduce it, given the DB/SWAPS files |
|
|
@jl777 Did you get anywhere with this? |
|
I get values, but I guess I need to use the exact SPV servers and not native mode? |
|
Interesting that it works in native mode. These are the Electrum servers I used: {
coin: 'KMD',
electrumServers: [
{
host: 'electrum1.cipig.net',
port: 10001,
},
{
host: 'electrum2.cipig.net',
port: 10001,
},
],
},
{
coin: 'CHIPS',
name: 'Chips',
rpcport: 57776,
pubtype: 60,
p2shtype: 85,
wiftype: 188,
txfee: 10000,
electrumServers: [
{
host: 'electrum1.cipig.net',
port: 10053,
},
{
host: 'electrum2.cipig.net',
port: 10053,
},
],
}, |
|
Finally found the problem! pushed fix to dev branch |
|
Awesome, thanks James! I think @artemii235 has set his repo to auto create a PR when you push to dev. So we should be able to get this in HyperDEX pretty soon. Will report back when I've tested. |
|
Dev branch was not updated yet, there is a PR waiting for merge. Automatic PR in my fork won't be created until this PR is merged. |
|
@jl777 Should this retroactively fix previous trades? If I call {
"uuid": "00b9fbebeab1b50b200f1fe631eb355bb03ff7d3d71ea6b94ccb18b984f3b3c4",
"expiration": 1530117529,
"tradeid": 2852472561,
"requestid": 3912469824,
"quoteid": 2093347877,
"iambob": 0,
"Bgui": "",
"Agui": "hyperdex",
"gui": "hyperdex",
"bob": "CHIPS",
"srcamount": 0.78639071,
"bobtxfee": 0.0001,
"alice": "KMD",
"destamount": 0.10009,
"alicetxfee": 0.00001,
"aliceid": "15837931641091850241",
"sentflags": [
"bobspend",
"alicepayment",
"bobdeposit",
"myfee",
"aliceclaim"
],
"values": [
0,
0,
0,
0.10011,
0.88488954,
0,
0.00012881,
0,
0,
0,
0
],
"result": "success",
"status": "finished",
"finishtime": 1531483660,
"bobdeposit": "8509fb343cc41c1d27abd288f0901e3fa34e69b5a381fa80ad40cbed16fe1857",
"alicepayment": "d29ba65bf9bd356434c1eadf6b0d54a4a9f09f73f5098d2eb4ac097c456f4366",
"bobpayment": "0000000000000000000000000000000000000000000000000000000000000000",
"paymentspent": "0000000000000000000000000000000000000000000000000000000000000000",
"Apaymentspent": "4c75699b58b046d67a0ce642d8014e9762706916813b3391b7b9c14ce98a6104",
"depositspent": "d96b723fae4da57c0a821e9375d428a0939d1dec91a19419b17a4d4ed813de4b",
"alicedexfee": "982ce4e35611039c9cdd6e340057497dc0b1deb1947f3827e58737ed55e74f4a",
"method": "tradestatus"
} |
|
if there is no .finished file, it should reprocess it and if it is fixed, it should work |
|
The above output is after a If so, it looks like it's not fixed. There's no |
In #764 you said this is the meaning of the values array:
However I have a trade with
aliceclaimand thealiceclaimvalue is not set in the array:The last item in
valuesis0when it should be0.88478954.You can see the
aliceclaimtx here: http://chips1.explorer.supernet.org/tx/d96b723fae4da57c0a821e9375d428a0939d1dec91a19419b17a4d4ed813de4bThe text was updated successfully, but these errors were encountered: