Merge pull request #100 from thiagobustamante/master

Fix Can\'t set headers after they are sent.
TreeGateway · Jan 25, 2018 · af9a797 · af9a797
2 parents 504f11c + 5574778
commit af9a797
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tree-gateway",
-  "version": "1.5.6",
+  "version": "1.5.7",
   "homepage": "http://treegateway.org",
   "description": "The Tree Gateway API Gateway",
   "author": "Thiago da Rosa de Bustamante <trbustamante@gmail.com>",

diff --git a/src/proxy/proxy.ts b/src/proxy/proxy.ts
@@ -146,7 +146,9 @@ export class ApiProxy {
             const errReference = 'https://nodejs.org/api/errors.html#errors_common_system_errors'; // link to Node Common Systems Errors page
 
             this.logger.error('[Tree-Gateway] Error occurred while trying to proxy request %s from %s to %s (%s) (%s)', req.url, hostname, target, err.code, errReference);
-            res.status(502).send('Bad Gateway');
+            if (!res.headersSent) {
+                res.status(502).send('Bad Gateway');
+            }
         });
 
         const maybeWrapResponse = this.interceptor.hasResponseInterceptor(api.proxy);