default content-type to html and json content type for json strings

suculent · Nov 19, 2023 · 91fd77a · 91fd77a
1 parent 0f55a8d
commit 91fd77a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/lib/router.auth.js b/lib/router.auth.js
@@ -240,6 +240,8 @@ module.exports = function (app) {
                     console.log("logoutAction error", err);
                 }
             });
+        } else {
+            console.log("[warning] logout attempt with undefined res.session!");
         }
         // Redirect to login page, must be on same CORS domain (public_url must be == public_url)...
         res.redirect(app_config.public_url);

diff --git a/lib/router.js b/lib/router.js
@@ -56,6 +56,9 @@ module.exports = function (app) {
 
   app.all("/*", function (req, res, next) {
 
+    // Default content-type, may be overridden later by Util.responder and others
+    res.header("Content-Type", "text/html; charset=utf-8");
+
     //
     // JWT Key Authentication
     //

diff --git a/lib/thinx/util.js b/lib/thinx/util.js
@@ -92,6 +92,7 @@ module.exports = class Util {
     } else if (typeOf(object) == "string") {
       res.end(object);
     } else {
+      if (typeof (res.header) === "function") res.header("Content-Type", "application/json; charset=utf-8");
       res.end(JSON.stringify(object));
     }
   }