Do not throw exception on double headers send. Just log error.

bem-node · May 26, 2015 · 1909368 · 1909368
1 parent dba9e1b
commit 1909368
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/blocks/i-response/i-response.priv.js b/blocks/i-response/i-response.priv.js
@@ -66,6 +66,12 @@ BEM.decl('i-response', null, {
      */
     redirect: function (path) {
         var res = this._getResponse();
+
+        if (res.finished) {
+            console.log('Cannot finish response. It is already finished.');
+            return;
+        }
+
         res.writeHead(302, {'Location': path});
         res.end();
     },
@@ -89,6 +95,11 @@ BEM.decl('i-response', null, {
         if (statusCode >= 500 && statusCode < 600) {
             console.error(err);
         }
+
+        if (res.finished) {
+            console.log('Cannot finish response. It is already finished.');
+        }
+
         res.writeHead(statusCode, err.message);
         res.end();
     },

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "bem-node",
-    "version": "0.9.5",
+    "version": "0.9.6",
     "description": "",
     "keywords": [
         "bem",