-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SSL and clickhouse cloud #10
Comments
@oatsandsugar this is not expected. I made a PR that allows for expanded set of options via env vars here: #11, along with some example configurations: https://github.com/ClickHouse/mcp-clickhouse/pull/11/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R143, would you mind seeing if that helps with this issue? If so, I'll land it. |
Thank you, I’ll review this evening
Johanan
…On Mon, Feb 17, 2025 at 4:51 PM Kaushik Iska ***@***.***> wrote:
@oatsandsugar <https://github.com/oatsandsugar> this is not expected. I
made a PR that allows for expanded set of options via env vars here: #11
<#11>, along with some
example configurations:
https://github.com/ClickHouse/mcp-clickhouse/pull/11/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R143,
would you mind seeing if that helps with this issue? If so, I'll land it.
—
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEGLDLYPN3P3RWC3FHENEWL2QJ7YHAVCNFSM6AAAAABXKLQN6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRUGMYDEMBZGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: iskakaushik]*iskakaushik* left a comment
(ClickHouse/mcp-clickhouse#10)
<#10 (comment)>
@oatsandsugar <https://github.com/oatsandsugar> this is not expected. I
made a PR that allows for expanded set of options via env vars here: #11
<#11>, along with some
example configurations:
https://github.com/ClickHouse/mcp-clickhouse/pull/11/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R143,
would you mind seeing if that helps with this issue? If so, I'll land it.
—
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEGLDLYPN3P3RWC3FHENEWL2QJ7YHAVCNFSM6AAAAABXKLQN6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRUGMYDEMBZGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Sorry, I commented on the PR, we got it working with verification set to false, but I reckon we should be able to get it working with verification set to true? Having got this working though, we are getting some incredible results. Happy to share our experiments. |
It would be amazing to see your experiments! It will be useful for me to pair with you to debug the SSL issue as well, could you send an email to |
Fixed by #15? |
More of a question than an issue.
I had no problem connecting to my local clickhouse, but when I tried to connect to clickhouse cloud I got this error
My gut is telling me that this is like when I was trying to connect to it with datagrip, and I had to toggle the USE SSL toggle, but not sure how I would do this in such a connection. Also, apologies if this isn't the best place to post this issue, I didn't see a community slack or anything like that
The text was updated successfully, but these errors were encountered: