X Tutup
Skip to content

Vouch Request: fix ClickHouse query client startup crash (#3184) #3198

@edosrecki

Description

@edosrecki

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      X Tutup