fix(gatsby-source-wordpress): check response exists before accessing …

…property (#11349)
gatsbyjs · Jan 28, 2019 · 3beb891 · 3beb891
1 parent 3742890
commit 3beb891
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/packages/gatsby-source-wordpress/src/__tests__/http-exception-handler.js b/packages/gatsby-source-wordpress/src/__tests__/http-exception-handler.js
@@ -0,0 +1,7 @@
+const httpExceptionHandler = require(`../http-exception-handler`)
+
+describe(`http-exception-handler`, () => {
+  it(`handles errors that lack responses without crashing`, () => {
+    expect(() => httpExceptionHandler({})).not.toThrowError()
+  })
+})
diff --git a/packages/gatsby-source-wordpress/src/http-exception-handler.js b/packages/gatsby-source-wordpress/src/http-exception-handler.js
@@ -7,12 +7,6 @@ const colorized = require(`./output-color`)
  */
 function httpExceptionHandler(e) {
   const { response, code } = e
-  console.log(
-    colorized.out(
-      `\nPath: ${response.request.path}`,
-      colorized.color.Font.FgRed
-    )
-  )
   if (!response) {
     console.log(
       colorized.out(
@@ -22,6 +16,12 @@ function httpExceptionHandler(e) {
     )
     return
   }
+  console.log(
+    colorized.out(
+      `\nPath: ${response.request.path}`,
+      colorized.color.Font.FgRed
+    )
+  )
   const {
     status,
     statusText,