symfony
diff --git a/‎src/Symfony/Component/HttpFoundation/Session/Storage/Handler/SessionHandlerFactory.php
Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/HttpFoundation/Session/Storage/Handler/SessionHandlerFactory.php
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/SessionHandlerFactoryTest.php
Lines changed: 30 additions & 1 deletion b/‎src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/SessionHandlerFactoryTest.php
Lines changed: 30 additions & 1 deletion
@@ -30,7 +30,7 @@ public static function createHandler($connection): AbstractSessionHandler
             throw new \TypeError(sprintf('Argument 1 passed to "%s()" must be a string or a connection object, "%s" given.', __METHOD__, get_debug_type($connection)));
         }
 
-        if ($options = parse_url($connection)) {
+        if ($options = \is_string($connection) ? parse_url($connection) : false) {
             parse_str($options['query'] ?? '', $options);
         }
 
 
@@ -12,6 +12,7 @@
 namespace Symfony\Component\HttpFoundation\Tests\Session\Storage\Handler;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\HttpFoundation\Session\Storage\Handler\RedisSessionHandler;
 use Symfony\Component\HttpFoundation\Session\Storage\Handler\SessionHandlerFactory;
 use Symfony\Component\HttpFoundation\Session\Storage\Handler\StrictSessionHandler;
 
@@ -28,7 +29,7 @@ class SessionHandlerFactoryTest extends TestCase
     /**
      * @dataProvider provideConnectionDSN
      */
-    public function testCreateHandler(string $connectionDSN, string $expectedPath, string $expectedHandlerType)
+    public function testCreateFileHandler(string $connectionDSN, string $expectedPath, string $expectedHandlerType)
     {
         $handler = SessionHandlerFactory::createHandler($connectionDSN);
 
@@ -45,4 +46,32 @@ public function provideConnectionDSN(): array
             'native file handler using provided save_path' => ['connectionDSN' => 'file://'.$base.'/session/storage', 'expectedPath' => $base.'/session/storage', 'expectedHandlerType' => StrictSessionHandler::class],
         ];
     }
+
+    /**
+     * @requires extension redis
+     */
+    public function testCreateRedisHandlerFromConnectionObject()
+    {
+        $handler = SessionHandlerFactory::createHandler($this->createMock(\Redis::class));
+        $this->assertInstanceOf(RedisSessionHandler::class, $handler);
+    }
+
+    /**
+     * @requires extension redis
+     */
+    public function testCreateRedisHandlerFromDsn()
+    {
+        $handler = SessionHandlerFactory::createHandler('redis://localhost?prefix=foo&ttl=3600&ignored=bar');
+        $this->assertInstanceOf(RedisSessionHandler::class, $handler);
+
+        $reflection = new \ReflectionObject($handler);
+
+        $prefixProperty = $reflection->getProperty('prefix');
+        $prefixProperty->setAccessible(true);
+        $this->assertSame('foo', $prefixProperty->getValue($handler));
+
+        $ttlProperty = $reflection->getProperty('ttl');
+        $ttlProperty->setAccessible(true);
+        $this->assertSame('3600', $ttlProperty->getValue($handler));
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public static function createHandler($connection): AbstractSessionHandler`
`30`	`30`	`throw new \TypeError(sprintf('Argument 1 passed to "%s()" must be a string or a connection object, "%s" given.', __METHOD__, get_debug_type($connection)));`
`31`	`31`	`}`
`32`	`32`
`33`		`- if ($options = parse_url($connection)) {`
	`33`	`+ if ($options = \is_string($connection) ? parse_url($connection) : false) {`
`34`	`34`	`parse_str($options['query'] ?? '', $options);`
`35`	`35`	`}`
`36`	`36`