feat: Reduce webhook response payload to status-only (calcom#25072)

* feat: reduce webhook payload

* Update packages/trpc/server/routers/viewer/webhook/testTrigger.handler.ts

Co-authored-by: Volnei Munhoz <volnei@cal.com>

---------

Co-authored-by: Volnei Munhoz <volnei@cal.com>

Author: hariombalhara

packages/features/webhooks/components/WebhookTestDisclosure.tsx

@@ -69,8 +69,12 @@ export default function WebhookTestDisclosure() {
         </div>
         <div className="bg-muted border-subtle rounded-b-lg border border-t-0 p-4 font-mono text-[13px] leading-4">
           {!mutation.data && <p>{t("no_data_yet")}</p>}
-          {mutation.status === "success" && (
-            <div className="overflow-x-auto">{JSON.stringify(mutation.data, null, 4)}</div>
+          {mutation.status === "success" && mutation.data && (
+            <div className="space-y-2">
+              <div>
+                <span className="text-subtle">{t("status")}:</span> <span className="text-emphasis">{mutation.data.status}</span>
+              </div>
+            </div>
           )}
         </div>
       </div>

packages/features/webhooks/lib/handleWebhookScheduledTriggers.ts

@@ -64,6 +64,8 @@ export async function handleWebhookScheduledTriggers(prisma: PrismaClient) {
         method: "POST",
         body: job.payload,
         headers,
+        // Avoid following redirect
+        redirect: "manual"
       }).catch((error) => {
         console.error(`Webhook trigger for subscriber url ${job.subscriberUrl} failed with error: ${error}`);
       })

packages/features/webhooks/lib/sendPayload.ts

@@ -305,16 +305,9 @@ const _sendPayload = async (
     body,
   });
 
-  const text = await response.text();
-
   return {
     ok: response.ok,
     status: response.status,
-    ...(text
-      ? {
-          message: text,
-        }
-      : {}),
   };
 };
 

packages/trpc/server/routers/viewer/webhook/testTrigger.handler.ts

@@ -43,12 +43,10 @@ export const testTriggerHandler = async ({ ctx: _ctx, input }: TestTriggerOption
   try {
     const webhook = { subscriberUrl: url, appId: null, payloadTemplate };
     return await sendPayload(secret, type, new Date().toISOString(), webhook, data);
-  } catch (_err) {
-    const error = getErrorFromUnknown(_err);
+  } catch {
     return {
       ok: false,
       status: 500,
-      message: error.message,
     };
   }
 };