Remove v1 GET api

Author: devksingh4

src/api/routes/membership.ts

@@ -226,133 +226,6 @@ const membershipPlugin: FastifyPluginAsync = async (fastify, _options) => {
           .send({ givenName, surname, netId, isPaidMember: false });
       },
     );
-    fastify.withTypeProvider<FastifyZodOpenApiTypeProvider>().get(
-      "/:netId",
-      {
-        schema: withTags(["Membership"], {
-          deprecated: true,
-          description: "[DEPRECATED] DO NOT USE!",
-          params: z.object({ netId: illinoisNetId }),
-          querystring: z.object({
-            list: z.string().min(1).optional().meta({
-              description:
-                "Membership list to check from (defaults to ACM Paid Member list).",
-            }),
-          }),
-          summary:
-            "Check ACM @ UIUC paid membership (or partner organization membership) status.",
-          response: {
-            200: {
-              description: "List membership status.",
-              content: {
-                "application/json": {
-                  schema: z
-                    .object({
-                      netId: illinoisNetId,
-                      list: z.optional(z.string().min(1)),
-                      isPaidMember: z.boolean(),
-                    })
-                    .meta({
-                      example: {
-                        netId: "rjjones",
-                        isPaidMember: false,
-                      },
-                    }),
-                },
-              },
-            },
-          },
-        }),
-      },
-      async (request, reply) => {
-        const netId = request.params.netId.toLowerCase();
-        const list = request.query.list || "acmpaid";
-        const cacheKey = `membership:${netId}:${list}`;
-        const result = await getKey<{ isMember: boolean }>({
-          redisClient: fastify.redisClient,
-          key: cacheKey,
-          logger: request.log,
-        });
-        if (result) {
-          return reply.header("X-ACM-Data-Source", "cache").send({
-            netId,
-            list: list === "acmpaid" ? undefined : list,
-            isPaidMember: result.isMember,
-          });
-        }
-        if (list !== "acmpaid") {
-          const isMember = await checkExternalMembership(
-            netId,
-            list,
-            fastify.dynamoClient,
-          );
-          await setKey({
-            redisClient: fastify.redisClient,
-            key: cacheKey,
-            data: JSON.stringify({ isMember }),
-            expiresIn: MEMBER_CACHE_SECONDS,
-            logger: request.log,
-          });
-          return reply.header("X-ACM-Data-Source", "dynamo").send({
-            netId,
-            list,
-            isPaidMember: isMember,
-          });
-        }
-        const isDynamoMember = await checkPaidMembershipFromTable(
-          netId,
-          fastify.dynamoClient,
-        );
-        if (isDynamoMember) {
-          await setKey({
-            redisClient: fastify.redisClient,
-            key: cacheKey,
-            data: JSON.stringify({ isMember: true }),
-            expiresIn: MEMBER_CACHE_SECONDS,
-            logger: request.log,
-          });
-          return reply
-            .header("X-ACM-Data-Source", "dynamo")
-            .send({ netId, isPaidMember: true });
-        }
-        const entraIdToken = await getEntraIdToken({
-          clients: await getAuthorizedClients(),
-          clientId: fastify.environmentConfig.AadValidClientId,
-          secretName: genericConfig.EntraSecretName,
-          logger: request.log,
-        });
-        const paidMemberGroup = fastify.environmentConfig.PaidMemberGroupId;
-        const isAadMember = await checkPaidMembershipFromEntra(
-          netId,
-          entraIdToken,
-          paidMemberGroup,
-        );
-        if (isAadMember) {
-          await setKey({
-            redisClient: fastify.redisClient,
-            key: cacheKey,
-            data: JSON.stringify({ isMember: true }),
-            expiresIn: MEMBER_CACHE_SECONDS,
-            logger: request.log,
-          });
-          reply
-            .header("X-ACM-Data-Source", "aad")
-            .send({ netId, isPaidMember: true });
-          await setPaidMembershipInTable(netId, fastify.dynamoClient);
-          return;
-        }
-        await setKey({
-          redisClient: fastify.redisClient,
-          key: cacheKey,
-          data: JSON.stringify({ isMember: false }),
-          expiresIn: MEMBER_CACHE_SECONDS,
-          logger: request.log,
-        });
-        return reply
-          .header("X-ACM-Data-Source", "aad")
-          .send({ netId, isPaidMember: false });
-      },
-    );
     fastify.withTypeProvider<FastifyZodOpenApiTypeProvider>().get(
       "/externalList",
       {

src/common/errors/index.ts

@@ -56,7 +56,7 @@ export class NotImplementedError extends BaseError<"NotImplementedError"> {
 
 export class UnauthorizedError extends BaseError<"UnauthorizedError"> {
   constructor({ message }: { message?: string }) {
-    super({ name: "UnauthorizedError", id: 101, message: message || "Caller does not have permission.", httpStatusCode: 401 });
+    super({ name: "UnauthorizedError", id: 101, message: message || "User does not have the privileges for this task.", httpStatusCode: 401 });
   }
 }