From 83099b909bed2fa1c5d205a803072d2d1232acb5 Mon Sep 17 00:00:00 2001 From: Abhay Date: Wed, 15 Apr 2026 17:56:50 +0530 Subject: [PATCH] fix: handle empty payloads in scheduled task test page (#3316) When parsePacket returns undefined for empty data (e.g. payload=''), getScheduleTaskRunPayload crashes accessing .timezone on undefined. Add a null guard that returns a failed SafeParse result so the run is gracefully skipped by the existing filter(Boolean) logic. --- apps/webapp/app/presenters/v3/TestTaskPresenter.server.ts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/apps/webapp/app/presenters/v3/TestTaskPresenter.server.ts b/apps/webapp/app/presenters/v3/TestTaskPresenter.server.ts index 09abb22639e..04f53442e10 100644 --- a/apps/webapp/app/presenters/v3/TestTaskPresenter.server.ts +++ b/apps/webapp/app/presenters/v3/TestTaskPresenter.server.ts @@ -382,6 +382,11 @@ export class TestTaskPresenter { async function getScheduleTaskRunPayload(payload: string, payloadType: string) { const packet = await parsePacket({ data: payload, dataType: payloadType }); + + if (!packet) { + return { success: false as const, error: new Error("Empty payload") }; + } + if (!packet.timezone) { packet.timezone = "UTC"; }