aminghadersohi commented on PR #44718:
URL: https://github.com/apache/superset/pull/44718#issuecomment-5894937054
@rusackas MariaDB gap: fixed in ac0680e2b4. `require_mysql_tls` now picks
options by DBAPI driver (optional `driver=` argument) instead of checking the
URL backend, and `MySQLEngineSpec` gates on an explicit `MYSQL_TLS_ENGINES =
{"mysql", "mariadb"}` allowlist. `mariadb://`, `mariadb+pymysql` and
`mariadb+mariadbconnector` are all enforced now. Doris calls
`require_mysql_tls(..., driver="mysqldb")` directly, and the fake-drivername
wrapper is gone. I left OceanBase and StarRocks out: their toggles still write
`ssl=0`, and StarRocks overrides `adjust_engine_params`. Either one can opt in
by joining the allowlist once someone checks it against its driver.
--
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]
