/cc @a1812 could you please have a look?

Looks like the code wants to use the URL path both for the unix scoket path and the DB index at the same time

I have the same issue, my env parameter: REDIS_DSN=redis://superpass@/run/redis/domain.local.sock?persistent=1
I was following https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html
v5.3.11 has this issue as well.

Anyone up for a PR fixing this?

I think the fix for the dbindex would be to look for it only if we have isset($params['host']) && isset($params['path']). Also, the !isset($params['dbindex'] looks weird to me, as $params is the output of parse_url

if we are talking about format like this redis://[email protected]?path=/another/socket/domain.sock?dbindex=5&persistent=1&something=else,
vendor/symfony/cache/Traits/RedisTrait.php
should be something like this:

and the final result would be:

However, I think this might be more complicated, need to test all other parameters and other variations
p.s. as @stof mentioned, some parts looks weird to me as well

based on https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html

and using format 'REDIS_DSN' => 'redis://superpass@/run/redis/domain.local.sock?persistent=1&anything=else/db-index=9',

looks like correct (uses 'pconnect' and follows documentation)

apologies for spamming (the previous example was wrongly interpreted by me), this should follow the documentation:

however, it's not covering situation when 'db-index' is not set

5.3.11 broken for redis://localhost:6379/ (OK with redis://localhost:6379 -> no ending slash)
was OK in 5.3.10

Invalid Redis DSN: "redis://localhost:6379/", the "dbindex" parameter must be a number.

We get the error in 5.3.12. Hasn't this been merged into the 5.3 branch?

The patch is in 5.3.12 I confirm.
Looks like #44232 might not be enough?
Please send a PR if you're affected and can figure out the proper fix.

Hi, @nicolas-grekas

This is the diff between 5.3.10 and 5.3.11 (5.3.10 works):

- if (!isset($params['dbindex']) && isset($params['path']) && preg_match('#/(\d+)$#', $params['path'], $m)) {	
-                $params['dbindex'] = $m[1];	
-                $params['path'] = substr($params['path'], 0, -\strlen($m[0]));
+ if (!isset($params['dbindex']) && isset($params['path'])) {
+                if (preg_match('#/(\d+)$#', $params['path'], $m)) {
+                    $params['dbindex'] = $m[1];
+                    $params['path'] = substr($params['path'], 0, -\strlen($m[0]));
+                } else {
+                    throw new InvalidArgumentException(sprintf('Invalid Redis DSN: "%s", the "dbindex" parameter must be a + number.', $dsn));
+                }

What was the reason for this change?

What is linked in this issue: see #43359

Okay. Could we reopen this issue?

I have the same problem as @loevgaard with 5.3.12 What is weird in one project it works OK, in another one it doesn't work, while on both of them I have symfony/cache 5.3.12
I use REDIS locally with MAMP PRO: CACHE_DSN='redis:///Applications/MAMP/tmp/redis.sock'

(removed)

The new problem is addressed here: #45277 (comment)

@thomas2411 @mathielen you're affected by #45277, not by this issue.

@loevgaard do you have more insights about your issue about dbindex, or does #45281 also fix you issue?

I am not experiencing this problem anymore, however, can't be quite sure because I don't quite remember how I triggered the problem (forgot to jot it down :()

OK, please report back with details when you can.

Okay, just got this error again: Invalid Redis DSN: "redis://127.0.0.1:51148/", the "dbindex" parameter must be a number..

It happens in here: https://github.com/symfony/symfony/blob/6.3/src/Symfony/Component/Cache/Traits/RedisTrait.php#L153.

The $dsn inputted in line 85 (https://github.com/symfony/symfony/blob/6.3/src/Symfony/Component/Cache/Traits/RedisTrait.php#L85) has this value: redis://127.0.0.1:51148/. An important thing to notice is that the environment variable isREDIS_URL=redis://127.0.0.1:51148 (notice the missing trailing slash) and just as a test I added this $dsn = trim($dsn, '/'); as the first line in the method and then everything works.

Can you send a PR if you have the fix please?

Yes, I will, @nicolas-grekas. Thanks!

There you have it, @nicolas-grekas. I tested it on the application that gave the error :)