lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix cash-out exchange behavior for 'rejected' tx

Browse source 
A TX on the 'rejected' state should still be watched for authorization.

This fixes the bug where delayed cash-out confirmations would not
trigger a exchange sell command.
This commit is contained in:
Rafael Taranto 2019-02-07 16:43:57 -03:00 • committed by Josh Harvey
parent df13f2ed90
commit 9364eb5132
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/cash-out/cash-out-atomic.js
View file

@ -119,10 +119,10 @@ function updateCassettes (t, tx) {
} }
function wasJustAuthorized (oldTx, newTx, isZeroConf) { function wasJustAuthorized (oldTx, newTx, isZeroConf) {
const isAuthorized = () => _.includes(oldTx.status, ['notSeen', 'published']) && const isAuthorized = () => _.includes(oldTx.status, ['notSeen', 'published', 'rejected']) &&
_.includes(newTx.status, ['authorized', 'instant', 'confirmed']) _.includes(newTx.status, ['authorized', 'instant', 'confirmed'])
const isConfirmed = () => _.includes(oldTx.status, ['notSeen', 'published', 'authorized']) && const isConfirmed = () => _.includes(oldTx.status, ['notSeen', 'published', 'authorized', 'rejected']) &&
_.includes(newTx.status, ['instant', 'confirmed']) _.includes(newTx.status, ['instant', 'confirmed'])
return isZeroConf ? isAuthorized() : isConfirmed() return isZeroConf ? isAuthorized() : isConfirmed()