Merge pull request #3643 from clairep94/pr05/migrate_routes_folder

pr05 Typescript Migration #11: Migrate server/routes folder

Author: raclim

package-lock.json

@@ -138,6 +138,7 @@
     "@types/friendly-words": "^1.2.2",
     "@types/jest": "^29.5.14",
     "@types/node": "^16.18.126",
+    "@types/passport": "^1.0.17",
     "@types/react": "^16.14.0",
     "@types/react-dom": "^16.9.25",
     "@types/react-router-dom": "^5.3.3",

package.json

@@ -2,7 +2,7 @@ import { Router } from 'express';
 import passport from 'passport';
 import * as ProjectController from '../controllers/project.controller';
 
-const router = new Router();
+const router = Router();
 
 router.get(
   '/:username/sketches',
@@ -24,4 +24,5 @@ router.delete(
   ProjectController.deleteProject
 );
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/api.routes.js renamed to server/routes/api.routes.ts

@@ -2,7 +2,7 @@ import { Router } from 'express';
 import { getProjectAsset } from '../controllers/project.controller';
 import { getFileContent } from '../controllers/file.controller';
 
-const router = new Router();
+const router = Router();
 
 router.get('/:username/sketches/:project_id/*', getProjectAsset);
 router.get('/full/:project_id/*', getProjectAsset);
@@ -14,4 +14,5 @@ router.get('/:username/embed/:project_id/*', getProjectAsset);
 
 router.get('/sketches/:project_id/assets/*?', getFileContent);
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/asset.routes.js renamed to server/routes/asset.routes.ts

@@ -2,7 +2,7 @@ import { Router } from 'express';
 import * as AWSController from '../controllers/aws.controller';
 import isAuthenticated from '../utils/isAuthenticated';
 
-const router = new Router();
+const router = Router();
 
 router.post('/S3/sign', isAuthenticated, AWSController.signS3);
 router.post(
@@ -17,4 +17,5 @@ router.delete(
 );
 router.get('/S3/objects', AWSController.listObjectsInS3ForUserRequestHandler);
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/aws.routes.js renamed to server/routes/aws.routes.ts

@@ -2,7 +2,7 @@ import { Router } from 'express';
 import * as CollectionController from '../controllers/collection.controller';
 import isAuthenticated from '../utils/isAuthenticated';
 
-const router = new Router();
+const router = Router();
 
 // List collections
 router.get(
@@ -41,4 +41,5 @@ router.delete(
   CollectionController.removeProjectFromCollection
 );
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/collection.routes.js renamed to server/routes/collection.routes.ts

@@ -2,7 +2,7 @@ import { Router } from 'express';
 
 const editorUrl = process.env.EDITOR_URL;
 
-const router = new Router();
+const router = Router();
 
 // CAT redirecting these temporarily to editor URLS to prevent phishing
 // router.get('/:username/embed/:project_id', EmbedController.serveProject);
@@ -21,4 +21,5 @@ router.get('/embed/:project_id', (req, res) => {
   res.redirect(301, `${editorUrl}/full/${projectId}`);
 });
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/embed.routes.js renamed to server/routes/embed.routes.ts

@@ -2,7 +2,7 @@ import { Router } from 'express';
 import * as FileController from '../controllers/file.controller';
 import isAuthenticated from '../utils/isAuthenticated';
 
-const router = new Router();
+const router = Router();
 
 router.post(
   '/projects/:project_id/files',
@@ -15,4 +15,5 @@ router.delete(
   FileController.deleteFile
 );
 
+// eslint-disable-next-line import/no-default-export
 export default router;

server/routes/file.routes.js renamed to server/routes/file.routes.ts

@@ -0,0 +1,44 @@
+import { Router, Request, Response, NextFunction } from 'express';
+import passport from 'passport';
+
+const router = Router();
+
+const authenticateOAuth = (service: string) => (
+  req: Request,
+  res: Response,
+  next: NextFunction
+) => {
+  passport.authenticate(
+    service,
+    { failureRedirect: '/login' },
+    (err: any, user: any) => {
+      if (err) {
+        // use query string param to show error;
+        res.redirect(`/account?error=${service}`);
+        return;
+      }
+
+      if (!user) {
+        res.redirect(`/account?error=${service}NoUser`);
+        return;
+      }
+
+      req.logIn(user, (loginErr) => {
+        if (loginErr) {
+          next(loginErr);
+          return;
+        }
+        res.redirect('/');
+      });
+    }
+  )(req, res, next);
+};
+
+router.get('/auth/github', passport.authenticate('github'));
+router.get('/auth/github/callback', authenticateOAuth('github'));
+
+router.get('/auth/google', passport.authenticate('google'));
+router.get('/auth/google/callback', authenticateOAuth('google'));
+
+// eslint-disable-next-line import/no-default-export
+export default router;