Skip to content

allow using ca_cert parameter - #466

Open
viktor-ozherelev wants to merge 2 commits into
ClickHouse:mainfrom
viktor-ozherelev:main
Open

allow using ca_cert parameter#466
viktor-ozherelev wants to merge 2 commits into
ClickHouse:mainfrom
viktor-ozherelev:main

Conversation

@viktor-ozherelev

@viktor-ozherelev viktor-ozherelev commented May 31, 2025

Copy link
Copy Markdown

Summary

Relates to the issue. This PR the problem of connecting to a Managed ClickHouse cluster whose server certificate is signed by an internal root Certificate Authority. Added support for the CA root in httpclient and nativeclient using the ca_cert parameter

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG

@CLAassistant

CLAassistant commented May 31, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@koletzilla
koletzilla self-requested a review August 7, 2026 14:36
@koletzilla

Copy link
Copy Markdown
Contributor

Hi @viktor-ozherelev , sorry for taking so long to check your PR.

I like the PR and it's ready to merge, but there're conflicts that I can't solve. The PR has the Maintainers are allowed to edit this pull request. tag, but your PR comes from your fork's main branch and main is a protected branch, so I can't do changes there.

Would you rebase and fix this issues? Just needs to change the entry of the changelog to the last release and leave the readme empty as now that docs lives in https://clickhouse.com/docs/integrations/connectors/data-ingestion/etl-tools/dbt. I'll update the docs with that.

Or if you prefer, I can recreate the branch, create a new PR with your commit cherrypicked (I think that way it keeps you as author) and merge it myself. I'd do that without problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants