eschutho commented on a change in pull request #14673:
URL: https://github.com/apache/superset/pull/14673#discussion_r633939747
##########
File path: superset/db_engine_specs/base.py
##########
@@ -1339,8 +1343,16 @@ class BasicParametersMixin:
"drivername://user:password@host:port/dbname[?key=value&key=value...]"
)
+ # query parameter to enable encryption in the database connection
+ # for Postgres this would be `{"sslmode": "verify-ca"}`, eg.
+ encryption_parameters: Dict[str, str] = {}
Review comment:
do you feel that we will need to check/enforce that this is defined on
the engine specs that use this mixin?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
