Skip to content

Commit f9f4535

Browse files
authored
Merge pull request #126 from wp-cli/fix/mariadb-ssl-verify
Explicitly enable `ssl-verify-server-cert` for mariadb
2 parents 74a3a1c + 1f4f743 commit f9f4535

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

.github/workflows/reusable-testing.yml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,8 @@ jobs:
122122
{
123123
"php": "8.3",
124124
"wp": "latest",
125-
"mysql": "mariadb-11.4"
125+
"mysql": "mariadb-11.4",
126+
"dbtype": "mariadb"
126127
},
127128
{
128129
"php": "7.2",
@@ -182,7 +183,8 @@ jobs:
182183
{
183184
"php": "8.3",
184185
"wp": "trunk",
185-
"mysql": "mariadb-11.4"
186+
"mysql": "mariadb-11.4",
187+
"dbtype": "mariadb"
186188
},
187189
{
188190
"php": "8.4",
@@ -386,6 +388,13 @@ jobs:
386388
my-cnf: |
387389
default_authentication_plugin=mysql_native_password
388390
391+
# Explicitly enable server certificate verification by default, avoiding
392+
# "option --ssl-verify-server-cert is disabled, because of an insecure passwordless login."
393+
# warnings because WP-CLI passes password via the `MYSQL_PWD` environment variable
394+
# and the setting is otherwise implicitly used.
395+
[client-mariadb]
396+
ssl-verify-server-cert
397+
389398
- name: Configure DB environment
390399
if: ${{ matrix.dbtype != 'sqlite' }}
391400
run: |

0 commit comments

Comments
 (0)