Merge pull request #29 from PurdueRCAC/fix-frontend

Fix JSON parse errors on frontend

Author: skkra0

backend/src/handlers/acceptInvitation.ts

@@ -21,5 +21,5 @@ export const acceptInvitationHandler: HandlerMap["acceptInvitation"] = async (
     throw e;
   }
 
-  return empty(200, res);
+  return empty(204, res);
 };

backend/src/handlers/claimOrg.ts

@@ -30,5 +30,5 @@ export const claimOrgHandler: HandlerMap["claimOrg"] = async (
     }
     throw e;
   }
-  return empty(200, res);
+  return empty(204, res);
 };

backend/src/handlers/createAppGroup.ts

@@ -18,7 +18,7 @@ export const createAppGroupHandler: HandlerMap["createAppGroup"] = async (
 
   try {
     await createAppGroup(req.user.id, data.orgId, data.name, data.apps);
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof AppCreateError) {
       const ex = e.cause!;

backend/src/handlers/deleteApp.ts

@@ -12,7 +12,7 @@ export const deleteAppHandler: HandlerMap["deleteApp"] = async (
   const appId = ctx.request.params.appId;
   try {
     await deleteApp(appId, req.user.id, ctx.request.requestBody.keepNamespace);
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof AppNotFoundError) {
       return json(404, res, { code: 404, message: "App not found" });

backend/src/handlers/deleteOrgByID.ts

@@ -18,5 +18,5 @@ export const deleteOrgByIDHandler: HandlerMap["deleteOrgByID"] = async (
     }
   }
 
-  return empty(200, res);
+  return empty(204, res);
 };

backend/src/handlers/githubWebhook.ts

@@ -35,13 +35,13 @@ export const githubWebhookHandler: HandlerMap["githubWebhook"] = async (
 
   try {
     await processGitHubWebhookPayload(requestType, action, JSON.parse(data));
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof ValidationError) {
       return json(400, res, { code: 400, message: e.message });
     } else if (e instanceof AppNotFoundError) {
       // GitHub sent a webhook about a repository, but it's not linked to any apps - nothing to do here
-      return empty(200, res);
+      return empty(204, res);
     } else if (e instanceof UnknownWebhookRequestTypeError) {
       // GitHub sent a webhook payload that we don't care about
       return empty(422, res);

backend/src/handlers/ingestLogs.ts

@@ -32,7 +32,7 @@ export const ingestLogsHandler: HandlerMap["ingestLogs"] = async (
       logType,
       ctx.request.requestBody.lines,
     );
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof DeploymentNotFoundError) {
       // No deployment matches the ID and secret

backend/src/handlers/setAppCD.ts

@@ -14,7 +14,7 @@ export const setAppCDHandler: HandlerMap["setAppCD"] = async (
       req.user.id,
       ctx.request.requestBody.enable,
     );
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof AppNotFoundError) {
       return json(404, res, { code: 404, message: "App not found." });

backend/src/handlers/updateApp.ts

@@ -15,7 +15,7 @@ export const updateAppHandler: HandlerMap["updateApp"] = async (
   const appData = ctx.request.requestBody;
   try {
     await updateApp(ctx.request.params.appId, req.user.id, appData);
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof AppNotFoundError) {
       return json(404, res, { code: 404, message: "App not found" });

backend/src/handlers/updateDeployment.ts

@@ -14,7 +14,7 @@ export const updateDeploymentHandler: HandlerMap["updateDeployment"] = async (
   const { secret, status } = ctx.request.requestBody;
   try {
     await updateDeployment(secret, status);
-    return empty(200, res);
+    return empty(204, res);
   } catch (e) {
     if (e instanceof ValidationError) {
       return json(404, res, { code: 400, message: e.message });