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

Documentation for PostgreSQL Metadata Store SSL Configuration Settings is incorrect #558

Open
navkumar opened this issue Jan 17, 2019 · 2 comments

Comments

@navkumar
Copy link

On this page in the documentation for the PostgreSQL Metadata Store (http://druid.io/docs/latest/development/extensions-core/postgresql.html), the settings table at the bottom uses incorrect properties that do not work in the latest (0.13.0-incubating) version of Druid.

According to the code here (PostgreSQLMetadataStorageModule.java), the SSL related settings are all prefixed with druid.metadata.postgres.ssl NOT druid.metadata.postgres as indicated on this documentation page. For example, you would enable SSL for PostgreSQL using the something like the following in runtime.properties:

...
# ALL THESE PROPERTIES ARE PREFIXED with "druid.metadata.postgres.ssl"
druid.metadata.postgres.ssl.enableSSL=true
druid.metadata.postgres.ssl.sslMode=verify-full
druid.metadata.postgres.ssl.sslRootCert=<path-to-ca.crt>
...

instead of:

...
# NONE OF THESE ARE READ BY THE POSTGRES METADATA STORAGE EXTENSION
druid.metadata.postgres.enableSSL=true
druid.metadata.postgres.sslMode=verify-full
druid.metadata.postgres.sslRootCert=<path-to-ca.crt>
...
@gianm
Copy link
Member

gianm commented Jan 17, 2019

Hey @navkumar, thanks for the catch. Our docs are stored in the https://github.com/apache/druid repo, under "docs". Would you be interested in raising a PR that fixes this?

@navkumar
Copy link
Author

Filed the pull request apache/druid#6884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants