search

trinodb / trino

Official repository of Trino, the distributed SQL query engine for big data, formerly known as PrestoSQL (https://trino.io)
https://trino.io
Apache License 2.0
10.41k stars 3k forks source link

Connection property 'SSLVerification' is not allowed in Trino JDBC Driver #14720

Closed jackwang2 closed 2 years ago

jackwang2 commented 2 years ago

Use the trino JDBC driver 400, seems it doesn't recognize property SLVerification mentioned in the Trino doc: Screen Shot 2022-10-24 at 16 23 13

mosabua commented 2 years ago

Could this parameter be missing in the validation of allowed parameters @electrum ?

Also @jackwang2 what application is this screenshot from? It looks like SSLVerification is not shown.. which could be a related problem

mosabua commented 2 years ago

It seems to be wired up correctly https://github.com/trinodb/trino/blob/master/client/trino-jdbc/src/main/java/io/trino/jdbc/ConnectionProperties.java#L104

Looking at your list of properties I think you might have an old jdbc driver on the classpath after all. Can you verify @jackwang2 ?

jackwang2 commented 2 years ago

@mosabua Do you mean old trino JDBC driver? I checked the variable java.class.path, it only contains one jar of trino-jdbc-400.jar.

java.sql.SQLException: Connection property 'SSLVerification' is not allowed
    at io.trino.jdbc.AbstractConnectionProperty.validate(AbstractConnectionProperty.java:139)
    at io.trino.jdbc.TrinoDriverUri.validateConnectionProperties(TrinoDriverUri.java:499)
    at io.trino.jdbc.TrinoDriverUri.<init>(TrinoDriverUri.java:133)
    at io.trino.jdbc.TrinoDriverUri.<init>(TrinoDriverUri.java:123)
    at io.trino.jdbc.TrinoDriverUri.create(TrinoDriverUri.java:144)
    at io.trino.jdbc.NonRegisteringTrinoDriver.connect(NonRegisteringTrinoDriver.java:53)
jackwang2 commented 2 years ago

Could this parameter be missing in the validation of allowed parameters @electrum ?

Also @jackwang2 what application is this screenshot from? It looks like SSLVerification is not shown.. which could be a related problem

The app is aqua data studio.

ebyhr commented 2 years ago

Please set SSL connection property as true.

mosabua commented 2 years ago

Ha! Good catch @ebyhr .. I totally missed that in the screenshot