fix: avoid calling fallback function twice (#198)

nodeshift · May 26, 2018 · d251820 · d251820
1 parent 1a61f56
commit d251820
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/circuit.js b/lib/circuit.js
@@ -355,10 +355,12 @@ class CircuitBreaker extends EventEmitter {
             }
           })
             .catch(error => {
-              this.semaphore.release();
-              const latencyEndTime = Date.now() - latencyStartTime;
-              handleError(
-                error, this, timeout, args, latencyEndTime, resolve, reject);
+              if (!timeoutError) {
+                this.semaphore.release();
+                const latencyEndTime = Date.now() - latencyStartTime;
+                handleError(
+                  error, this, timeout, args, latencyEndTime, resolve, reject);
+              }
             });
         } catch (error) {
           this.semaphore.release();