mirror of https://github.com/apache/superset.git
fix: change sslmode to require for Postgres (#15642)
* change sslmode to require * fix test
This commit is contained in:
parent
2252f3396c
commit
f39582c900
|
@ -186,7 +186,7 @@ class PostgresEngineSpec(PostgresBaseEngineSpec, BasicParametersMixin):
|
||||||
"postgresql://user:password@host:port/dbname[?key=value&key=value...]"
|
"postgresql://user:password@host:port/dbname[?key=value&key=value...]"
|
||||||
)
|
)
|
||||||
# https://www.postgresql.org/docs/9.1/libpq-ssl.html#LIBQ-SSL-CERTIFICATES
|
# https://www.postgresql.org/docs/9.1/libpq-ssl.html#LIBQ-SSL-CERTIFICATES
|
||||||
encryption_parameters = {"sslmode": "verify-ca"}
|
encryption_parameters = {"sslmode": "require"}
|
||||||
|
|
||||||
max_column_name_length = 63
|
max_column_name_length = 63
|
||||||
try_remove_schema_from_table_name = False
|
try_remove_schema_from_table_name = False
|
||||||
|
|
|
@ -460,7 +460,7 @@ def test_base_parameters_mixin():
|
||||||
)
|
)
|
||||||
assert sqlalchemy_uri == (
|
assert sqlalchemy_uri == (
|
||||||
"postgresql+psycopg2://username:password@localhost:5432/dbname?"
|
"postgresql+psycopg2://username:password@localhost:5432/dbname?"
|
||||||
"foo=bar&sslmode=verify-ca"
|
"foo=bar&sslmode=require"
|
||||||
)
|
)
|
||||||
|
|
||||||
parameters_from_uri = PostgresEngineSpec.get_parameters_from_uri(sqlalchemy_uri)
|
parameters_from_uri = PostgresEngineSpec.get_parameters_from_uri(sqlalchemy_uri)
|
||||||
|
|
Loading…
Reference in New Issue