fix(@opentelemetry/exporter-collector): remove fulfilled promises cor…

…rectly
open-telemetry · Dec 22, 2020 · 6bb67cf · 6bb67cf
1 parent a2304c9
commit 6bb67cf
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/...ges/opentelemetry-exporter-collector/src/platform/browser/CollectorExporterBrowserBase.ts b/...ges/opentelemetry-exporter-collector/src/platform/browser/CollectorExporterBrowserBase.ts
@@ -68,19 +68,14 @@ export abstract class CollectorExporterBrowserBase<
     const serviceRequest = this.convert(items);
     const body = JSON.stringify(serviceRequest);
 
-    const promise = new Promise(resolve => {
+    const promise = new Promise<void>((resolve) => {
       const _onSuccess = (): void => {
         onSuccess();
-        _onFinish();
+        resolve();
       };
       const _onError = (error: collectorTypes.CollectorExporterError): void => {
         onError(error);
-        _onFinish();
-      };
-      const _onFinish = () => {
         resolve();
-        const index = this._sendingPromises.indexOf(promise);
-        this._sendingPromises.splice(index, 1);
       };
 
       if (this._useXHR) {
@@ -97,5 +92,10 @@ export abstract class CollectorExporterBrowserBase<
       }
     });
     this._sendingPromises.push(promise);
+    const _onFinish = () => {
+      const index = this._sendingPromises.indexOf(promise);
+      this._sendingPromises.splice(index, 1);
+    };
+    promise.then(_onFinish);
   }
 }