Merge pull request #18 from clout-stack/task/fix-promise-handling

fix promise handling
clout-stack · Feb 2, 2020 · 8db384b · 8db384b
2 parents ce42431 + df37bfd
commit 8db384b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -1,6 +1,7 @@
 language: node_js
 sudo: false
 node_js:
+  - "12.0.0"
   - "10.0.0"
   - "8.12.0"
 install:

diff --git a/hookslib/apiType/api.js b/hookslib/apiType/api.js
@@ -10,7 +10,8 @@ function handlePromiseAPIDef(fn) {
   return function postPromiseHandle(req, resp, next, ...args) {
     safePromisifyCallFn(fn, this, [req, resp, null, ...args])
       .then((data) => {
-        if (!resp.headerSent) {
+        const headerSent = !!resp._header;
+        if (!headerSent) {
           return resp.ok(data);
         }
       })

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "clout-js",
-  "version": "2.1.0-beta.6",
+  "version": "2.1.0-beta.7",
   "description": "Clean, simplistic, enterprise grade full-stack NodeJS framework",
   "main": "index.js",
   "scripts": {