Write special env variables at the very end of the installer

[Boy132]

Closes #696

This makes sure that "special" env variables (in this case the SESSION_DRIVER) are only written to the .env file at the very end of the installer.
The SESSION_DRIVER variable is the only variable that directly affects the installer when being changed.

[rmartinoscar]

panel/app/Filament/Pages/Installer/PanelInstaller.php

Line 74 in 382dcb3

->hidden(fn (Get $get) => $get('env_general.SESSION_DRIVER') != 'redis' && $get('env_general.QUEUE_CONNECTION') != 'redis' && $get('env_general.CACHE_STORE') != 'redis'),

->hidden(fn (Get $get) => $get('env_special.SESSION_DRIVER') != 'redis' && $get('env_general.QUEUE_CONNECTION') != 'redis' && $get('env_special.CACHE_STORE') != 'redis'), 

panel/app/Filament/Pages/Installer/Steps/EnvironmentStep.php

Line 63 in 382dcb3

ToggleButtons::make('env_general.CACHE_STORE')

ToggleButtons::make('env_special.CACHE_STORE') 

We have to change those too

[Boy132]

panel/app/Filament/Pages/Installer/PanelInstaller.php

Line 74 in 382dcb3

->hidden(fn (Get $get) => $get('env_general.SESSION_DRIVER') != 'redis' && $get('env_general.QUEUE_CONNECTION') != 'redis' && $get('env_general.CACHE_STORE') != 'redis'),

->hidden(fn (Get $get) => $get('env_special.SESSION_DRIVER') != 'redis' && $get('env_general.QUEUE_CONNECTION') != 'redis' && $get('env_special.CACHE_STORE') != 'redis'), 

panel/app/Filament/Pages/Installer/Steps/EnvironmentStep.php

Line 63 in 382dcb3

ToggleButtons::make('env_general.CACHE_STORE')

ToggleButtons::make('env_special.CACHE_STORE') 

We have to change those too

Hidden check is fixed. And the cache driver is fine, it doesn't need special treatment.

[rmartinoscar]

I had the same 500 as before when hitting next in the db step with redis selected as cache store it errors immediately i can't even fill in the redis host & pass so it uses no password cause no redis env is set yet.
This is fixed with my proposed changes.

Using redis which is on something else but localhost will also throw

[Boy132]

I had the same 500 as before when hitting next in the db step with redis selected as cache store it errors immediately i can't even fill in the redis host & pass so it uses no password cause no redis env is set yet. This is fixed with my proposed changes.

Using redis which is on something else but localhost will also throw

Please try again and if you still get a 500 post the error.
The error you linked is also from the session driver and not the cache.

[rmartinoscar]

I had the same 500 as before when hitting next in the db step with redis selected as cache store it errors immediately i can't even fill in the redis host & pass so it uses no password cause no redis env is set yet. This is fixed with my proposed changes.

Using redis which is on something else but localhost will also throw

Please try again and if you still get a 500 post the error. The error you linked is also from the session driver and not the cache.

https://paste.pelistuff.com/kin4
I only selected cache store in the first step

[lancepioch]

We should validate the Redis connection too and make sure it works before saving it in the .env, good catch!

[Boy132]

I had the same 500 as before when hitting next in the db step with redis selected as cache store it errors immediately i can't even fill in the redis host & pass so it uses no password cause no redis env is set yet. This is fixed with my proposed changes.

Using redis which is on something else but localhost will also throw

Please try again and if you still get a 500 post the error. The error you linked is also from the session driver and not the cache.

https://paste.pelistuff.com/kin4 I only selected cache store in the first step

Yeah so that's a totally different error...

We should validate the Redis connection too and make sure it works before saving it in the .env, good catch!

The redis connection is validated - in the Redis step.

The problem is the SoftwareVersionService that gets called when we run the config clear command. (I assume that's because of dependency injection)