-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Open
Description
Why do you want to contribute?
I'm setting up a self-hosted Trigger.dev instance using the Docker Compose "combined setup" and ran into a startup crash caused by a missing url.searchParams.delete("secure") call in initializeQueryClickhouseClient() (apps/webapp/app/services/clickhouseInstance.server.ts). This is the same issue reported in #3184.
The fix is a one-liner that makes this function consistent with its two sibling functions (initializeClickhouseClient and initializeLogsClickhouseClient), which already strip the secure query parameter before passing the URL to the ClickHouse client library.
I have the fix ready and would like to open a PR once vouched.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels