Merge pull request #1583 from SynBioHub/fixSecureSetup

Check if https used for setup
SynBioHub · Dec 3, 2024 · cbb710c · cbb710c
2 parents 6d69488 + 7d0511e
commit cbb710c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/app.js b/lib/app.js
@@ -145,6 +145,7 @@ function App () {
 
   // use cors
   app.use(cors())
+  app.set('trust proxy', true)
 
   const uploadToMemory = multer({
     storage: multer.memoryStorage({})

diff --git a/lib/views/setup.js b/lib/views/setup.js
@@ -58,8 +58,8 @@ module.exports = function (req, res) {
   const settings = {
     instanceName: 'My SynBioHub',
     frontendURL: '',
-    instanceUrl: req.secure ? req.protocol + 's' : req.protocol + '://' + req.get('Host') + '/',
-    uriPrefix: req.secure ? req.protocol + 's' : req.protocol + '://' + req.get('Host') + '/',
+    instanceUrl: (req.secure ? req.protocol + 's' : req.protocol) + '://' + req.get('Host') + '/',
+    uriPrefix: (req.secure ? req.protocol + 's' : req.protocol) + '://' + req.get('Host') + '/',
     altHome: '',
     userName: '',
     affiliation: '',