add TODO

acm-uiuc · Nov 12, 2024 · 076d1b8 · 076d1b8
1 parent 2655a80
commit 076d1b8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -56,9 +56,9 @@ async function init() {
       app.secretValue =
         (await getSecretValue(genericConfig.ConfigSecretName)) || {};
     }
-    // if (!app.sequelizeInstance) {
-    //   app.sequelizeInstance = await getSequelizeInstance(app);
-    // }
+    if (!app.sequelizeInstance) {
+      app.sequelizeInstance = await getSequelizeInstance(app);
+    }
     req.startTime = now();
     req.log.info({ url: req.raw.url }, "received request");
   });

diff --git a/src/routes/linkry.ts b/src/routes/linkry.ts
@@ -264,6 +264,7 @@ const linkryRoutes: FastifyPluginAsync = async (fastify, _options) => {
         throw new UnauthorizedError({ message: "Could not get user roles." });
       }
       try {
+        // TODO: optimize this to use a proper query in sequelize
         const isAdmin = request.userRoles.has(AppRoles.LINKS_ADMIN);
         let filteredLinks = await ShortLinkModel.findAll();
         if (!isAdmin) {