fix(core): always skip paths with non html content-types

Fixes #96
harlan-zw · May 13, 2023 · 2c753b6 · 2c753b6
1 parent 790d856
commit 2c753b6
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/core/src/puppeteer/tasks/html.ts b/packages/core/src/puppeteer/tasks/html.ts
@@ -20,10 +20,8 @@ export const extractHtmlPayload: (page: Page, route: string) => Promise<{ succes
       return { success: false, message: `Invalid response from URL ${route} code: ${response?.status || '404'}.` }
 
     // ignore non-html
-    if (response.headers['content-type'] && !response.headers['content-type'].includes('text/html')) {
-      if (!valid || !response)
-        return { success: false, message: `Invalid response from URL ${route} content type: ${response.headers['content-type']}.` }
-    }
+    if (response.headers['content-type'] && !response.headers['content-type'].includes('text/html'))
+      return { success: false, message: `Non-HTML Content-Type header: ${response.headers['content-type']}.` }
 
     return {
       success: true,