[12.x] Update schema dump to use new MySQL SSL disable flag #55758
+10
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#55683 allowed SSL to be disabled when Laravel imports schema SQL. However, the
--ssl=off
flag was deprecated in 8.0.26 and removed in MySQL 8.0.40:Error: [ERROR] unknown variable 'ssl=off'.
This PR checks the MySQL version and uses the newer
--ssl-mode=DISABLED
for newer versions.https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-26.html#mysqld-8-0-26-deprecation-removal
https://dev.mysql.com/doc/relnotes/mysql/8.4/en/news-8-4-0.html#mysqld-8-4-0-deprecation-removal
https://dev.mysql.com/doc/refman/8.4/en/connection-options.html#option_general_ssl-mode
An alternative to this might be a generic
LARAVEL_LOAD_OPTS=
env var that allows any custom options/flags to be added to the load command.