fix redirect loop

qbreader · Oct 12, 2024 · 7520187 · 7520187
1 parent 3cca6c9
commit 7520187
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -23,6 +23,8 @@ if (process.env.NODE_ENV !== 'production') {
   app.use(morgan('dev'));
 }
 
+// https://stackoverflow.com/questions/10348906/how-to-know-if-a-request-is-http-or-https-in-node-js
+app.enable('trust proxy');
 app.use(hostnameRedirection);
 app.use(httpsEnforcement);
 

diff --git a/server/https-enforcement.js b/server/https-enforcement.js
@@ -3,7 +3,7 @@ export default function httpsEnforcement (req, res, next) {
   console.log(req.protocol);
 
   // Use HTTPS if not on localhost
-  if (req.protocol !== 'https' && !['localhost', '127.0.0.1'].includes(hostname)) {
+  if (!req.secure && !['localhost', '127.0.0.1'].includes(hostname)) {
     return res.redirect(301, `https://${hostname}${req.originalUrl}`);
   }