chore(auth): remove unnecessary changes

Author: jescalada

src/service/index.js

@@ -64,16 +64,10 @@ const createApp = async () => {
   app.use(express.json());
   app.use(express.urlencoded({ extended: true }));
   app.use('/', routes);
-
-  // In production, serves the static files from the build directory
-  if (process.env.NODE_ENV === 'production') {
-    app.use('/', express.static(absBuildPath));
-    app.get('/*', (req, res) => {
-      res.sendFile(path.join(`${absBuildPath}/index.html`));
-    });
-  } else {
-    console.log('Not serving static files');
-  }
+  app.use('/', express.static(absBuildPath));
+  app.get('/*', (req, res) => {
+    res.sendFile(path.join(`${absBuildPath}/index.html`));
+  });
 
   return app;
 };

src/service/routes/auth.js

@@ -3,13 +3,7 @@ const router = new express.Router();
 const passport = require('../passport').getPassport();
 const db = require('../../db');
 const passportType = passport.type;
-const { GIT_PROXY_UI_HOST: uiHost = 'http://localhost', NODE_ENV } = process.env;
-
-// TODO: Refactor this through proper .env loading. This handles redirects in dev.
-let uiPort = 3000;
-if (NODE_ENV === 'production') {
-  uiPort = process.env.GIT_PROXY_UI_PORT;
-}
+const { GIT_PROXY_UI_HOST: uiHost = 'http://localhost', GIT_PROXY_UI_PORT: uiPort = 3000 } = process.env;
 
 router.get('/', (req, res) => {
   res.status(200).json({