change error messaging for paymentservice

splunk · May 31, 2024 · 37a035b · 37a035b
1 parent 53bcbf8
commit 37a035b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/paymentservice/charge.js b/src/paymentservice/charge.js
@@ -59,7 +59,8 @@ module.exports.charge = async request => {
       const randomNumber = Math.floor(Math.random() * 1000) + 1;
       if (randomNumber <= failRate) {  
         let errorMsg="PaymentService charge failed. Bad API token (simulated).";
-        logger.error({transactionId, cardType, lastFourDigits, amount: { units, nanos, currencyCode }}, errorMsg);
+        //Already getting logged when the error is caught.
+        //logger.error({transactionId, cardType, lastFourDigits, amount: { units, nanos, currencyCode }}, errorMsg);
         throw new Error(errorMsg);
       }
   }

diff --git a/src/paymentservice/index.js b/src/paymentservice/index.js
@@ -22,7 +22,7 @@ async function chargeServiceHandler(call, callback) {
     callback(null, response)
 
   } catch (err) {
-    logger.warn({ err })
+    logger.error({ err })
 
     span.recordException(err)
     span.setStatus({ code: opentelemetry.SpanStatusCode.ERROR })