-
Notifications
You must be signed in to change notification settings - Fork 514
Description
Sail Version
1.47.0
Laravel Version
12.37
PHP Version
8.4
Operating System
Linux
OS Version
22
Description
On a fresh installation by https://laravel.build/app?with=mysql,
APP_PORT and similar env var does not work as expected.
Fresh installation does not include APP_PORT=<80> in .env file.
If in compose.yml file services -> laravel.test -> ports
I set '${APP_PORT:-84}:80'
Now if APP_PORT is not defined in .env file then APP_PORT should have value 84
But it does not work, and it takes 80 which is defined in file "vendor/laravel/sail/bin/sail"
This is not ideal, it should not use default value from vendor files, but instead it should use 84
Current workaround is setting APP_PORT=84 .env file and default value in compose file has no effect.
So I propose to add APP_PORT=80 in .env file and remove default port in compose.yml file or else some changes should be made in vendor/laravel/sail/bin/sail file
Steps To Reproduce
Mentioned above in description