From 3efcdf515ac174052040f767a7569b2730032cd4 Mon Sep 17 00:00:00 2001 From: Osrecki Date: Mon, 9 Mar 2026 20:54:06 +0100 Subject: [PATCH] fix(webapp): strip secure param from query ClickHouse URL The `initializeQueryClickhouseClient()` function was missing the `url.searchParams.delete("secure")` call that the other two sibling ClickHouse client init functions already had. This caused a startup crash (`Error: Unknown URL parameters: secure`) when QUERY_CLICKHOUSE_URL fell back to CLICKHOUSE_URL which contains `?secure=false`. Fixes #3184 --- .server-changes/fix-clickhouse-query-client-secure-param.md | 6 ++++++ apps/webapp/app/services/clickhouseInstance.server.ts | 3 +++ 2 files changed, 9 insertions(+) create mode 100644 .server-changes/fix-clickhouse-query-client-secure-param.md diff --git a/.server-changes/fix-clickhouse-query-client-secure-param.md b/.server-changes/fix-clickhouse-query-client-secure-param.md new file mode 100644 index 0000000000..4daa021fe4 --- /dev/null +++ b/.server-changes/fix-clickhouse-query-client-secure-param.md @@ -0,0 +1,6 @@ +--- +area: webapp +type: fix +--- + +Strip `secure` query parameter from QUERY_CLICKHOUSE_URL before passing to ClickHouse client. This was already done for the main and logs ClickHouse clients but was missing for the query client, causing a startup crash with `Error: Unknown URL parameters: secure`. diff --git a/apps/webapp/app/services/clickhouseInstance.server.ts b/apps/webapp/app/services/clickhouseInstance.server.ts index f88b3baaae..61494811a0 100644 --- a/apps/webapp/app/services/clickhouseInstance.server.ts +++ b/apps/webapp/app/services/clickhouseInstance.server.ts @@ -83,6 +83,9 @@ function initializeQueryClickhouseClient() { const url = new URL(env.QUERY_CLICKHOUSE_URL); + // Remove secure param + url.searchParams.delete("secure"); + return new ClickHouse({ url: url.toString(), name: "query-clickhouse",