fix: ensure onError is only called once

open-telemetry · Aug 9, 2024 · aef7d15 · aef7d15
1 parent 831e170
commit aef7d15
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/experimental/packages/otlp-exporter-base/src/platform/browser/OTLPExporterBrowserBase.ts b/experimental/packages/otlp-exporter-base/src/platform/browser/OTLPExporterBrowserBase.ts
@@ -98,15 +98,13 @@ export abstract class OTLPExporterBrowserBase<
       .then(response => {
         if (response.status === 'success') {
           onSuccess();
-          return;
-        }
-        if (response.status === 'failure' && response.error) {
+        } else if (response.status === 'failure' && response.error) {
           onError(response.error);
-        }
-        if (response.status === 'retryable') {
+        } else if (response.status === 'retryable') {
           onError(new OTLPExporterError('Export failed with retryable status'));
+        } else {
+          onError(new OTLPExporterError('Export failed with unknown error'));
         }
-        onError(new OTLPExporterError('Export failed with unknown error'));
       }, onError);
 
     this._sendingPromises.push(promise);