BTCPrivate
/
z-nomp
mirror of https://github.com/BTCPrivate/z-nomp.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix payment redis final cleanout issue - possible double spending - optimized

This commit is contained in:
Eugene@ubuntu 2014-04-02 14:56:13 +04:00
parent 6aae77a159
commit 7fe215679a
1 changed files with 27 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
libs/paymentProcessor.js
View File

@ -493,6 +493,33 @@ function SetupForPool(logger, poolOptions, setupFinished){
});
},
/* Call redis to check if previous sendmany and/or redis cleanout commands completed successfully.
If sendmany worked fine but redis commands failed you HAVE TO run redis commands again
(manually) to prevent double payments. If sendmany failed too you can safely delete
coin + '_finalRedisCommands' string from redis to let pool calculate payments again. */
function(magnitude, workerPayments, finalRedisCommands, callback) {
redisClient.get(coin + '_finalRedisCommands', function(error, reply) {
if (error){
callback('Check finished - error with redis getting finalRedisCommands' + JSON.stringify(error));
return;
}
if (reply) {
callback('Check finished - previous sendmany and/or redis cleanout commands failed - ' + reply);
return;
} else {
/* There was no error in previous sendmany and/or redis cleanout commands
so we can safely continue */
redisClient.set(coin + '_finalRedisCommands', JSON.stringify(finalRedisCommands), function(error, reply) {
if (error){
callback('Check finished - error with saving finalRedisCommands' + JSON.stringify(error));
return;
}
callback(null, magnitude, workerPayments, finalRedisCommands);
});
}
});
},
function(magnitude, workerPayments, finalRedisCommands, callback){
//This does the final all-or-nothing atom transaction if block deamon sent payments