move check within handler

ctrlplanedev · Oct 9, 2024 · b1323bd · b1323bd
1 parent eda7513
commit b1323bd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/apps/webservice/src/app/api/github/webhook/route.ts b/apps/webservice/src/app/api/github/webhook/route.ts
@@ -9,12 +9,12 @@ import { env } from "~/env";
 import { handleCommitWebhookEvent } from "./commit/handler";
 import { handleWorkflowWebhookEvent } from "./workflow/handler";
 
-const secret = env.GITHUB_WEBHOOK_SECRET;
-if (secret == null) throw new Error("GITHUB_WEBHOOK_SECRET is not set");
-const webhooks = new Webhooks({ secret });
-
 export const POST = async (req: NextRequest) => {
   try {
+    const secret = env.GITHUB_WEBHOOK_SECRET;
+    if (secret == null) throw new Error("GITHUB_WEBHOOK_SECRET is not set");
+    const webhooks = new Webhooks({ secret });
+
     const signature = req.headers.get("x-hub-signature-256")?.toString();
     if (signature == null)
       return new NextResponse("No signature", { status: 401 });