This PR allows developers to indepdently set ssl_ca, ssl_cert and ssl_key.
Not all SSL connections require a key, cert and CA file. For instance, Azure Database for MySQL enables SSL connections with only a CA file. See: https://docs.microsoft.com/en-us/azure/mysql/howto-configure-ssl This commit allows developers to port M1 databases out of Azure Database for MySQL to another environment without compromising security.
Fixed Issues (if relevant)
N/A
Manual testing scenarios
Try to create a connection to an Azure Database for MySQL by setting ssl_ca in the database config.
Run bin/magento migrate:settings
Fail with SQLSTATE[HY000] [9002] SSL connection is required. Please specify SSL options and retry.
Contribution checklist
[x] Pull request has a meaningful description of its purpose
[x] All commits are accompanied by meaningful commit messages
[x] All new or changed code is covered with unit/integration tests (if applicable)
Hi @damienwebdev, thank you for your contribution! Please, complete Contribution Survey, it will take less than a minute. Your feedback will help us to improve contribution process.
Description
This PR allows developers to indepdently set
ssl_ca
,ssl_cert
andssl_key
.Not all SSL connections require a key, cert and CA file. For instance, Azure Database for MySQL enables SSL connections with only a CA file. See: https://docs.microsoft.com/en-us/azure/mysql/howto-configure-ssl This commit allows developers to port M1 databases out of Azure Database for MySQL to another environment without compromising security.
Fixed Issues (if relevant)
N/A
Manual testing scenarios
ssl_ca
in the database config.bin/magento migrate:settings
SQLSTATE[HY000] [9002] SSL connection is required. Please specify SSL options and retry.
Contribution checklist