diff --git a/xmlrpc-server.js b/xmlrpc-server.js index c8762cb..e4d744a 100644 --- a/xmlrpc-server.js +++ b/xmlrpc-server.js @@ -151,7 +151,7 @@ function composeXmlRpcResponse(param) { } } - logger.info(JSON.stringify(data)); + //logger.info(JSON.stringify(data)); return xml2jsBuilder.buildObject(data); } @@ -296,7 +296,7 @@ function sendReply(response) { }); } -function errorOnTopUpReport(topUpReportError) { +function errorOnTopUpReport(topUpReportError, debugInfo) { var hasError = false; if (!topUpReportError) { @@ -305,14 +305,14 @@ function errorOnTopUpReport(topUpReportError) { try { if (topUpReportError.body) { - logger.verbose('errorOnTopUpReport response body: ' + topUpReportError.body); + logger.verbose('errorOnTopUpReport response body: ' + topUpReportError.body, {req: debugInfo}); } - if (topUpReportError.res && topUpReportError.res.statusCode) { - logger.verbose('Partner response with status code:' + topUpReportError.res.statusCode); + if (topUpReportError.res && topUpReportError.res.statusCode && topUpReportError.res.statusCode != '200') { + logger.verbose('Partner response with status code:' + topUpReportError.res.statusCode, {req: debugInfo}); if (topUpReportError.res.statusCode != '200') { - logger.warn('Going to resend topUpReport because status code is not 200 (' + topUpReportError.res.statusCode + ')'); + logger.warn('Going to resend topUpReport because status code is not 200 (' + topUpReportError.res.statusCode + ')', {req: debugInfo}); return true; } } @@ -380,7 +380,7 @@ function sendTopUpReport(reverseUrls, params, urlIdx, retry) { client.methodCall(methodName, [ params ], function (topUpReportError, value) { - if (errorOnTopUpReport(topUpReportError)) { + if (errorOnTopUpReport(topUpReportError, {params: params, clientOptions: clientOptions})) { logger.warn('Error sending topUpReport retrying another url (if available): ' + topUpReportError, {error: topUpReportError.toString()}); sendTopUpReport(reverseUrls, params, ++urlIdx, retry);