aminghadersohi commented on PR #44736:
URL: https://github.com/apache/superset/pull/44736#issuecomment-5905636319
@fitzee thanks for the thorough check. Both notes are addressed in
1da7d19f96, after merging master in 5bd233a8c3.
1. SSL switch: the `encryption_parameters` comment and the Doris docs
metadata (engine spec `notes` and `databases.json`) now say that `VERIFY_CA`
also verifies the hostname with MariaDB Connector/C. The switch needs
`ssl_ca=<path>` in Additional parameters and an FE certificate matching the
host, so a default self-signed FE fails closed.
2. Denylist: UPDATING.md notes that hiding Doris now takes `{"pydoris":
{"mysqldb"}}`. `test_denylist_matches_the_default_driver` covers the new key
and the old one.
StarRocks is left for a separate change.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
