[13.x] Ability to set session prefix for Redis

[jackbayliss]

On Redis providers like Upstash / clusters / ElastiCache and general managed redis services, you're limited to a single keystore (db0), so you can't isolate sessions from cache by using a separate store.

Sessions currently inherit the cache prefix by default with setups like:

SESSION_DRIVER=redis
CACHE_STORE=redis
SCHEDULE_CACHE_DRIVER=redis

This makes it extremely difficult to scan and figure out what's what, between cache and sessions. Meaning when you need to clear cache manually or clear space its complex.

This PR allows you to set a prefix, and is completely opt in, which mirrors the session.connection option.
it could be defaulted, but i'm not sure or in charge of those executive decisions. But, maybe this should be the default in 14.x to prevent confusion?

// config/session.php
'prefix' => env('SESSION_PREFIX', Str::slug((string) env('APP_NAME', 'laravel')).'-session-'),

I can move this to 14.x if you'd prefer too

Alternatively, we can do something like the below, but it was a lot less discoverable and confusing, which is why I thought of this PR.

// .env
// SESSION_STORE = sessions

// config/cache.php
  'sessions' => [
          'driver' => 'redis',
          'prefix' => '-session-',   
],

This PR will also assist #60705

[github-actions]

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

[taylorotwell]

Hmm, if we set the prefix here would it override or stomp on the cache prefix. Is that object shared between the two?

[jackbayliss]

@taylorotwell I think its OK cause its cloned

framework/src/Illuminate/Session/SessionManager.php

Line 181 in 56a36ae

clone $this->container->make('cache')->store($store),

So, they are both seperate at least that's how I read it :D (I did test it locally and it worked)