-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Ensure db commands work with env:cli
#10490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried adding LOCAL_PHP="8.4-fpm" to my .env (after having downgraded to 8.2-fpm due to this issue), and after a restart, I got the failure again on trunk:
$ npm run env:cli db export
> [email protected] env:cli
> node ./tools/local-env/scripts/docker.js exec --user wp_php cli wp db export
Warning: Failed to get current character set of the posts table. Reason: ERROR 2026 (HY000): TLS/SSL error: self-signed certificate in certificate chain
mariadb-dump: Got error: 2026: "TLS/SSL error: self-signed certificate in certificate chain" when trying to connect
Upon switching to this branch, the issue is fixed!
$ npm run env:cli db export
> [email protected] env:cli
> node ./tools/local-env/scripts/docker.js exec --user wp_php cli wp db export
Success: Exported to 'wordpress_develop-2025-11-07-bcb8816.sql'.
Thanks!
|
For me current in trunk export working fine. |
In r60735, a change was made to the installation-related code in the local development environment that invokes any
wp dbcommands to circumvent a failure caused by the presence of a self-signed certificate within the certificate chain.However, that did not fix any
wp dbcommands called through theenv:clinpm script. This aims to do that by adding--defaultsanytimeenv:cliis run with a command nested underdb.Trac ticket: Core-64218.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.