BACKEND-11: Fix auth issues

cuappdev · Sep 12, 2024 · d9042b1 · d9042b1
1 parent 8c0ee18
commit d9042b1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -10,9 +10,6 @@ import { authMiddleware } from "./middleware/authMiddleware";
 const app = express();
 app.use(bodyParser.json());
 
-// Define auth middleware first
-app.use(authMiddleware);
-
 // Default Route
 app.get("/", (req, res) => {
   return res.sendFile("index.html", { root: __dirname });
@@ -24,6 +21,10 @@ app.use("/api-docs", swaggerUI.serve, async (req: Request, res: Response) => {
     swaggerUI.generateHTML(await import("../build/swagger.json"))
   );
 });
+
+// Define auth middleware first
+app.use(authMiddleware);
+
 RegisterRoutes(app);
 
 // Define error middleware last

diff --git a/src/middleware/authMiddleware.ts b/src/middleware/authMiddleware.ts
@@ -8,7 +8,7 @@ export const authMiddleware = async (
   res: Response,
   next: NextFunction
 ): Promise<Response | void> => {
-  // Exclude getActiveAnnouncements route
+  // Exclude getActiveAnnouncements and API docs route
   const routePattern = /^\/announcements\/\w+/;
   if (routePattern.test(req.path)) {
     return next();