Skip to content
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

Clickhouse destination allows turning off SSL #21507

Closed
grishick opened this issue Jan 18, 2023 · 1 comment · Fixed by #21509
Closed

Clickhouse destination allows turning off SSL #21507

grishick opened this issue Jan 18, 2023 · 1 comment · Fixed by #21509
Labels
needs-triage team/destinations Destinations team's backlog type/bug Something isn't working

Comments

@grishick
Copy link
Contributor

grishick commented Jan 18, 2023

What

Clickhouse destination in cloud does not hide "SSL Connection" toggle. The code in destination-clickhouse-strict-encrypt looks like it should hide it, but for whatever reason, it does not:
clickhouse

@grishick grishick added type/bug Something isn't working needs-triage team/destinations Destinations team's backlog labels Jan 18, 2023
@grishick
Copy link
Contributor Author

I think the bug is in build.gradle:

application {
    mainClass = 'io.airbyte.integrations.destination.clickhouse.ClickhouseDestination'
    applicationDefaultJvmArgs = ['-XX:+ExitOnOutOfMemoryError', '-XX:MaxRAMPercentage=75.0']
}

should be:

application {
    mainClass = 'io.airbyte.integrations.destination.clickhouse.ClickhouseDestinationStrictEncrypt'
    applicationDefaultJvmArgs = ['-XX:+ExitOnOutOfMemoryError', '-XX:MaxRAMPercentage=75.0']
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage team/destinations Destinations team's backlog type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant