check that content type exists before setting it

gethomepage · Sep 5, 2022 · 08afa0b · 08afa0b
1 parent bad436b
commit 08afa0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/utils/proxies/credentialed.js b/src/utils/proxies/credentialed.js
@@ -19,7 +19,7 @@ export default async function credentialedProxyHandler(req, res) {
         },
       });
 
-      res.setHeader("Content-Type", contentType);
+      if (contentType) res.setHeader("Content-Type", contentType);
       return res.status(status).send(data);
     }
   }

diff --git a/src/utils/proxies/generic.js b/src/utils/proxies/generic.js
@@ -12,7 +12,7 @@ export default async function genericProxyHandler(req, res) {
       const url = new URL(formatApiCall(widget.type, { endpoint, ...widget }));
       const [status, contentType, data] = await httpProxy(url);
 
-      res.setHeader("Content-Type", contentType);
+      if (contentType) res.setHeader("Content-Type", contentType);
       return res.status(status).send(data);
     }
   }
-Original file line number
+Diff line change
@@ Expand Up @@
             },
           });
-          res.setHeader("Content-Type", contentType);
+          if (contentType) res.setHeader("Content-Type", contentType);
           return res.status(status).send(data);
         }
       }
@@ Expand Down @@