Commit bdd6c81f8bf55b85c590c1d4e3f25731f099154b
1 parent
2568a36cbc
Exists in
master
again
Showing 1 changed file with 15 additions and 8 deletions Side-by-side Diff
xmlrpc-server.js
... | ... | @@ -344,16 +344,23 @@ function sendTopUpReport(reverseUrls, params, urlIdx, retry) { |
344 | 344 | var methodName = 'topUpReport'; |
345 | 345 | logger.info('Requesting topUpReport', {params: params}); |
346 | 346 | client.methodCall(methodName, [ params ], function (error, value) { |
347 | - | |
348 | - if (error) { | |
349 | - if ((error.constructor !== Object) || (Object.keys(error).length > 0)) { | |
350 | - logger.warn('Error sending topUpReport retrying another url (if available)', {error: error, value: value}); | |
351 | - sendTopUpReport(reverseUrls, params, ++urlIdx, retry); | |
352 | - return; | |
347 | + | |
348 | + try { | |
349 | + | |
350 | + if (error) { | |
351 | + if ((error.constructor !== Object) || (Object.keys(error).length === 0)) { | |
352 | + logger.warn('Error sending topUpReport retrying another url (if available)', {error: error, value: value}); | |
353 | + sendTopUpReport(reverseUrls, params, ++urlIdx, retry); | |
354 | + return; | |
355 | + } | |
353 | 356 | } |
354 | - } | |
355 | 357 | |
356 | - logger.verbose("topUpReport ACK", {error: error, value: value}); | |
358 | + logger.verbose("topUpReport ACK", {error: error, value: value}); | |
359 | + | |
360 | + } | |
361 | + catch(topUpReportException) { | |
362 | + logger.warn("Exception on sending topUpReport" {e: topUpReportException}); | |
363 | + } | |
357 | 364 | |
358 | 365 | }); |
359 | 366 | } |