Merge pull request #282 from Azure-Samples/fix-issue-with-port

Fix issue with port is undefined in hybrid SPA sample

Author: salman90

6-AdvancedScenarios/4-sign-in-hybrid/App/controllers/authController.js

@@ -1,5 +1,6 @@
 require("dotenv").config();
 const msalInstance = require("../msal");
+const port = process.env.PORT || 5000;
 
 exports.loginUser = async (req, res, next) => {
   const authCodeUrlParameters = {
@@ -30,7 +31,7 @@ exports.handleRedirectWithCode = (req, res, next) => {
     enableSpaAuthorizationCode: true,
     scopes: ["User.Read"],
   };
-
+  
   msalInstance
     .acquireTokenByCode(tokenRequest)
     .then((response) => {
@@ -48,13 +49,7 @@ exports.handleRedirectWithCode = (req, res, next) => {
       req.session.referredUsername = preferredUsername;
       req.session.authenticated = true;
 
-      const urlFrom = (urlObject) =>
-        String(
-          Object.assign(
-            new URL(`http://localhost:${process.env.port ? port : 5000}`),
-            urlObject
-          )
-        );
+      const urlFrom = (urlObject) => String(Object.assign(new URL(`http://localhost:${port}`), urlObject));
 
       res.redirect(
         urlFrom({