Skip to content

Add basic TLS configuration info to JDBC connector docs#10668

Merged
hashhar merged 1 commit intotrinodb:masterfrom
jhlodin:jl/global-tls-jdbc
Jan 21, 2022
Merged

Add basic TLS configuration info to JDBC connector docs#10668
hashhar merged 1 commit intotrinodb:masterfrom
jhlodin:jl/global-tls-jdbc

Conversation

@jhlodin
Copy link
Copy Markdown
Contributor

@jhlodin jhlodin commented Jan 18, 2022

We have little-to-no information on the connector docs suggesting that you can secure the connection between Trino and the data source. This PR adds some high level information (the main JDBC URL parameter, use with globally-trusted certificates, example connection-url property) and links out to the corresponding JDBC driver docs for more information.

@cla-bot cla-bot bot added the cla-signed label Jan 18, 2022
@jhlodin jhlodin added the docs label Jan 18, 2022
@jhlodin jhlodin force-pushed the jl/global-tls-jdbc branch 3 times, most recently from 67e5f6d to b9bd7b3 Compare January 18, 2022 21:47
@jhlodin jhlodin force-pushed the jl/global-tls-jdbc branch from b9bd7b3 to aced895 Compare January 18, 2022 22:24
Copy link
Copy Markdown
Contributor

@m57lyra m57lyra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jhlodin jhlodin force-pushed the jl/global-tls-jdbc branch from aced895 to 0dc4d0c Compare January 19, 2022 19:27
@jhlodin jhlodin force-pushed the jl/global-tls-jdbc branch from 0dc4d0c to 9b30b1c Compare January 20, 2022 16:13
@jhlodin jhlodin requested a review from hashhar January 20, 2022 16:13
@jhlodin jhlodin force-pushed the jl/global-tls-jdbc branch from 9b30b1c to cf5452e Compare January 20, 2022 21:17
@hashhar hashhar merged commit 5313c35 into trinodb:master Jan 21, 2022
@github-actions github-actions bot added this to the 369 milestone Jan 21, 2022
@jhlodin jhlodin deleted the jl/global-tls-jdbc branch January 21, 2022 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants