Skip to content

Support application-supplied SSLContext in client-v2 and jdbc-v2#2918

Merged
chernser merged 4 commits into
mainfrom
polyglot/jdbc-v2-custom-ssl-context
Jul 17, 2026
Merged

Support application-supplied SSLContext in client-v2 and jdbc-v2#2918
chernser merged 4 commits into
mainfrom
polyglot/jdbc-v2-custom-ssl-context

Reject non-SSLContext ssl_context value at point of use in createHttp…

7409993
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 17, 2026 in 1m 5s

Annotations

Check warning on line 1197 in client-v2/src/main/java/com/clickhouse/client/api/Client.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 70 to 64, Complexity from 16 to 14, Nesting Level from 4 to 2, Number of Variables from 11 to 6.

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ9xhWCIZYIwyl1GQ12b&open=AZ9xhWCIZYIwyl1GQ12b&pullRequest=2918

Check warning on line 360 in jdbc-v2/src/main/java/com/clickhouse/jdbc/internal/JdbcConfiguration.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof SSLContext sslcontext'

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ9HqWvgVWej1L9_isBY&open=AZ9HqWvgVWej1L9_isBY&pullRequest=2918

Check failure on line 1458 in client-v2/src/main/java/com/clickhouse/client/api/Client.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Failed to get response. Server returned {}. Retrying. (Duration: {})" 3 times.

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ9xhWCIZYIwyl1GQ12a&open=AZ9xhWCIZYIwyl1GQ12a&pullRequest=2918

Check failure on line 3 in client-v2/src/test/java/com/clickhouse/client/api/internal/HttpAPIClientHelperTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add some tests to this class.

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ9xhV93ZYIwyl1GQ12Z&open=AZ9xhV93ZYIwyl1GQ12Z&pullRequest=2918

Check warning on line 150 in client-v2/src/main/java/com/clickhouse/client/api/Client.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Constructor has 8 parameters, which is greater than 7 authorized.

See more on https://sonarcloud.io/project/issues?id=ClickHouse_clickhouse-java&issues=AZ9HqWtfVWej1L9_isBX&open=AZ9HqWtfVWej1L9_isBX&pullRequest=2918