feat(http): add response headers also to reject path

Useful e.g. for status code 401 Unauthorized, which must include a WWW-Authenticate header field.
cerebral · Jan 3, 2017 · 2505530 · 2505530
1 parent 014f7ee
commit 2505530
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/packages/cerebral-provider-http/README.md b/packages/cerebral-provider-http/README.md
@@ -178,6 +178,7 @@ function someGetAction ({http}) {
     .catch((response) => {
       response.status // Status code of response
       response.result // Parsed response text
+      response.headers // Parsed response headers
     })
 }
 ```

diff --git a/packages/cerebral-provider-http/src/DEFAULT_OPTIONS.js b/packages/cerebral-provider-http/src/DEFAULT_OPTIONS.js
@@ -29,10 +29,11 @@ export default {
       result = JSON.parse(xhr.responseText)
     }
 
+    const responseHeaders = 'getAllResponseHeaders' in xhr
+      ? parseHeaders(xhr.getAllResponseHeaders())
+      : null
+
     if (xhr.status >= 200 && xhr.status < 300) {
-      const responseHeaders = 'getAllResponseHeaders' in xhr
-        ? parseHeaders(xhr.getAllResponseHeaders())
-        : null
       resolve({
         status: xhr.status,
         headers: responseHeaders,
@@ -41,6 +42,7 @@ export default {
     } else {
       reject({
         status: xhr.status,
+        headers: responseHeaders,
         result: result
       })
     }