In https://github.com/tighten/takeout/pull/330, we reverted back to the official MySQL image. After merging that PR, I noticed that it now complains about the --default-authentication-plugin flag that we are setting saying it doesn't exist (from https://github.com/tighten/takeout/pull/116). I don't see that same flag being set on Laravel Sail. After testing without it, I was able to connect from my host machine, then created a sample Laravel Sail project to use the Takeout MySQL container, and it works. So we're dropping it.
The MYSQL_ALLOW_EMPTY_PASSWORD flag can always be set to 1 instead of making it dynamic based on the password used.
Changed
--default-authentication-plugin
flag that we are setting saying it doesn't exist (from https://github.com/tighten/takeout/pull/116). I don't see that same flag being set on Laravel Sail. After testing without it, I was able to connect from my host machine, then created a sample Laravel Sail project to use the Takeout MySQL container, and it works. So we're dropping it.MYSQL_ALLOW_EMPTY_PASSWORD
flag can always be set to1
instead of making it dynamic based on the password used.