fix: removed retry logic from v1 cm360 proxy handler

rudderlabs · Dec 1, 2023 · d786189 · d786189
1 parent a015460
commit d786189
Showing 1 changed file with 2 additions and 23 deletions.
diff --git a/src/v1/destinations/campaign_manager/networkHandler.js b/src/v1/destinations/campaign_manager/networkHandler.js
@@ -10,25 +10,6 @@ const {
 } = require('../../../adapters/utils/networkUtils');
 const tags = require('../../../v0/util/tags');
 
-function isEventRetryableAndExtractErrMsg(element, proxyOutputObj) {
-  let isRetryable = false;
-  let errorMsg = '';
-  // success event
-  if (!element.errors) {
-    return isRetryable;
-  }
-  for (const err of element.errors) {
-    errorMsg += `${err.message}, `;
-    if (err.code === 'INTERNAL') {
-      isRetryable = true;
-    }
-  }
-  if (errorMsg) {
-    proxyOutputObj.error = errorMsg;
-  }
-  return isRetryable;
-}
-
 function isEventAbortableAndExtractErrMsg(element, proxyOutputObj) {
   let isAbortable = false;
   let errorMsg = '';
@@ -68,10 +49,8 @@ const responseHandler = (destinationResponse) => {
         metadata: rudderJobMetadata[idx],
         error: 'success',
       };
-      // update status of partial event as per retriable or abortable
-      if (isEventRetryableAndExtractErrMsg(element, proxyOutputObj)) {
-        proxyOutputObj.statusCode = 500;
-      } else if (isEventAbortableAndExtractErrMsg(element, proxyOutputObj)) {
+      // update status of partial event if abortable
+      if (isEventAbortableAndExtractErrMsg(element, proxyOutputObj)) {
         proxyOutputObj.statusCode = 400;
       }
       responseWithIndividualEvents.push(proxyOutputObj);