-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
undefined index db-ssl-verify #26762
Comments
Hi @DanielRuf. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @DanielRuf do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
I'm working on it. |
Hi @DanielRuf. Thank you for working on this issue.
|
Hi, I had the same problem while configuring Redis using Magento Dev Docs as a guide.
|
I don't think that my ddev setup uses redis. So your issue might be a different one. |
Have you tried to declare those index in setup? I had to do that as well for each command line I used. The only difference is that I made the setup using wizard. You can also try to declare in env.php like this:
Otherwise, you can declare in your config:setup (any of them) . --db . flags for each of those indexes (1007, 1008, 1009 and 1014) |
I mean a very specific field which fails in a standard setup (ddev). I did nothing special. And not sure what these numbers are for. I have already opened a PR with a bugfix for such cases. |
I don't this is a bug. 1007, 1008, 1009 and 1014 each one correspond to --db-ssl-verify --db-cert-verify, the problem which your are facing while configuring setup. So, after configuring setup, db-ssl-verify turns out to be 1007 in driver options inside your env.php file. |
This seems to be already set in the generated file. Normal setup and so on worked but not the specific command which should nothing have to do with the field (just change one setting). The change in the PR prevents this error. |
I'm not from Magento team, I'm just sharing with you how I solved a similar error calling config:setup. I did some research on that new feature and I didn't open an issue. I just said for you to try to declare those index while using config:setup. The same values that are declared in your env.php. I just think it is missing in Dev Docs files. So, you PR is relevant if you think so, but then it is up to Magento Administrators accept it or not according to their milestones and plans. |
I install Magento 2.3.4 using composer:
then I run
after that I have absolutely fresh installation and newly created
Then I have this warning:
every time I run
even w/o options. |
Both PRs will make it more stable and prevent such cases or errors that are not catched and cause issues in a later step. |
Hi @engcom-Alfa. Thank you for working on this issue.
|
✅ Confirmed by @engcom-Alfa Issue Available: @engcom-Alfa, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Thanks @gcampedelli ! magento setup:config:set \
--db-ssl-key="" \
--db-ssl-cert="" \
--db-ssl-ca=/path/tho/ca.pem \
--db-ssl-verify \
--session-save=db |
Hi @DanielRuf. Thank you for your report. The fix will be available with the upcoming 2.4.0 release. |
Hi @DanielRuf. Thank you for your report.
The fix will be available with the upcoming 2.4.2 release. |
Preconditions (*)
Steps to reproduce (*)
Follow the install instructions at https://ddev.readthedocs.io/en/latest/users/cli-usage/#magento-2-quickstart and then try to change the backend frontend name using the ssh container.
Expected result (*)
Actual result (*)
The text was updated successfully, but these errors were encountered: