From 819850142774f6ebc999ff8d68440aeced1b0aa0 Mon Sep 17 00:00:00 2001 From: Rup Sarmah <84681565+sarmah-rup@users.noreply.github.com> Date: Fri, 2 Oct 2026 11:47:12 +0530 Subject: [PATCH] fix(importers): keep Checkly followRedirects on imported monitors (#2834) --- packages/importers/src/providers/checkly/mapper.test.ts | 5 +++++ packages/importers/src/providers/checkly/mapper.ts | 1 + packages/services/src/import/phase-writers.ts | 2 ++ 3 files changed, 8 insertions(+) diff --git a/packages/importers/src/providers/checkly/mapper.test.ts b/packages/importers/src/providers/checkly/mapper.test.ts index f7123a6b..5c32a5e6 100644 --- a/packages/importers/src/providers/checkly/mapper.test.ts +++ b/packages/importers/src/providers/checkly/mapper.test.ts @@ -114,6 +114,11 @@ describe("mapCheck", () => { expect(m.active).toBe(false); expect(m.method).toBe("POST"); }); + + test("carries over the check's followRedirects setting", () => { + expect(mapCheck(MOCK_CHECKS[0], 1).followRedirects).toBe(true); + expect(mapCheck(MOCK_CHECKS[3], 1).followRedirects).toBe(false); + }); }); describe("deriveSlug", () => { diff --git a/packages/importers/src/providers/checkly/mapper.ts b/packages/importers/src/providers/checkly/mapper.ts index 46da5d97..87102d6a 100644 --- a/packages/importers/src/providers/checkly/mapper.ts +++ b/packages/importers/src/providers/checkly/mapper.ts @@ -125,6 +125,7 @@ export function mapCheck(check: ChecklyCheck, workspaceId: number) { headers, body: req?.body ?? "", method: mapMethod(req?.method ?? "GET"), + followRedirects: req?.followRedirects ?? true, // Checkly maxResponseTime is the hard timeout (ms); default to 45s. timeout: check.maxResponseTime ?? 45000, sourceMonitorGroupId: check.groupId, diff --git a/packages/services/src/import/phase-writers.ts b/packages/services/src/import/phase-writers.ts index 8122d196..6ab1ef1c 100644 --- a/packages/services/src/import/phase-writers.ts +++ b/packages/services/src/import/phase-writers.ts @@ -726,6 +726,7 @@ export async function writeMonitorsPhase( body: string; method: string; timeout: number; + followRedirects?: boolean; sourceMonitorGroupId: string | null; }; @@ -793,6 +794,7 @@ export async function writeMonitorsPhase( | "CONNECT" | "OPTIONS", timeout: data.timeout, + followRedirects: data.followRedirects, }) .returning(); -- 2.51.2