Skip to content

Add SSL verification control for ClickHouse handler (continue of #11367)#11842

Merged
ZoranPandovski merged 4 commits into
developfrom
fix_ssl_clickhouse
Nov 11, 2025
Merged

Add SSL verification control for ClickHouse handler (continue of #11367)#11842
ZoranPandovski merged 4 commits into
developfrom
fix_ssl_clickhouse

Conversation

@ea-rus
Copy link
Copy Markdown
Collaborator

@ea-rus ea-rus commented Nov 1, 2025

Description

Continuer of #11367

Fixed

  • verify is True by default and is optional
  • simplify way to compose url

Fixes https://linear.app/mindsdb/issue/FQE-1464/bug-clickhouse-handler-problem-with-self-signed-certificate-in

Type of change

  • 🐛 Bug fix (non-breaking change which fixes an issue)

Verification Process

To ensure the changes are working as expected:

  • Test Location: Specify the URL or path for testing.
  • Verification Steps: Outline the steps or queries needed to validate the change. Include any data, configurations, or actions required to reproduce or see the new functionality.

Additional Media:

  • I have attached a brief loom video or screenshots showcasing the new functionality or change.

Checklist:

  • My code follows the style guidelines(PEP 8) of MindsDB.
  • I have appropriately commented on my code, especially in complex areas.
  • Necessary documentation updates are either made or tracked in issues.
  • Relevant unit and integration tests are updated or added.

kaizenjinco and others added 3 commits November 1, 2025 22:37
Co-authored-by: Bao Ngo <baonn@cbbank.vn>
Co-authored-by: entelligence-ai-pr-reviews[bot] <174136889+entelligence-ai-pr-reviews[bot]@users.noreply.github.com>
@ZoranPandovski ZoranPandovski merged commit caaa0c1 into develop Nov 11, 2025
23 checks passed
@ZoranPandovski ZoranPandovski deleted the fix_ssl_clickhouse branch November 11, 2025 14:40
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants