Merge branch '5.4' into 6.0

* 5.4: (46 commits)
  move username/password fix to non-deprecated Connection class
  cs fix
  [VarDumper] Fix dumping twig templates found in exceptions
  do not replace definition arguments that have not been configured
  fix Console tests on Windows
  [Validator] Add translations for CIDR constraint
  [Dotenv] Fix testLoadEnv() to start from a fresh context
  [Console] Add completion to server:dump command
  bug #42194 [RateLimiter] fix: sliding window policy to use microtime
  [Validator] Update validators.sr_Cyrl.xlf
  [Validator] Update validators.sr_Latn.xlf
  Add suggestions for the option 'format' of lints commands: twig, yaml and xliff
  [VarDumper] Add support for Fiber
  uzb translation
  Update validators.uz.xlf
  Fix logging of impersonator introduced in 5.3
  [Console] Add show proxified command class in completion debug
  skip command completion tests with older Symfony Console versions
  [Uid] Allow use autocompletion
  [Console] Add completion to messenger:setup-transports command
  ...

Author: nicolas-grekas

CHANGELOG.md

@@ -25,6 +25,7 @@ CHANGELOG
  * Deprecate passing `null` as `$requestIp` to `IpUtils::__checkIp()`, `IpUtils::__checkIp4()` or `IpUtils::__checkIp6()`, pass an empty string instead.
  * Add the `litespeed_finish_request` method to work with Litespeed
  * Deprecate `upload_progress.*` and `url_rewriter.tags` session options
+ * Allow setting session options via DSN
 
 5.3
 ---

Session/Storage/Handler/MemcachedSessionHandler.php

@@ -38,19 +38,19 @@ class MemcachedSessionHandler extends AbstractSessionHandler
      *
      * List of available options:
      *  * prefix: The prefix to use for the memcached keys in order to avoid collision
-     *  * expiretime: The time to live in seconds.
+     *  * ttl: The time to live in seconds.
      *
      * @throws \InvalidArgumentException When unsupported options are passed
      */
     public function __construct(\Memcached $memcached, array $options = [])
     {
         $this->memcached = $memcached;
 
-        if ($diff = array_diff(array_keys($options), ['prefix', 'expiretime'])) {
+        if ($diff = array_diff(array_keys($options), ['prefix', 'expiretime', 'ttl'])) {
             throw new \InvalidArgumentException(sprintf('The following options are not supported "%s".', implode(', ', $diff)));
         }
 
-        $this->ttl = isset($options['expiretime']) ? (int) $options['expiretime'] : 86400;
+        $this->ttl = $options['expiretime'] ?? $options['ttl'] ?? null;
         $this->prefix = $options['prefix'] ?? 'sf2s';
     }
 
@@ -69,7 +69,7 @@ protected function doRead(string $sessionId): string
 
     public function updateTimestamp(string $sessionId, string $data): bool
     {
-        $this->memcached->touch($this->prefix.$sessionId, time() + $this->ttl);
+        $this->memcached->touch($this->prefix.$sessionId, time() + (int) ($this->ttl ?? ini_get('session.gc_maxlifetime')));
 
         return true;
     }
@@ -79,7 +79,7 @@ public function updateTimestamp(string $sessionId, string $data): bool
      */
     protected function doWrite(string $sessionId, string $data): bool
     {
-        return $this->memcached->set($this->prefix.$sessionId, $data, time() + $this->ttl);
+        return $this->memcached->set($this->prefix.$sessionId, $data, time() + (int) ($this->ttl ?? ini_get('session.gc_maxlifetime')));
     }
 
     /**

Session/Storage/Handler/SessionHandlerFactory.php

@@ -23,6 +23,12 @@ class SessionHandlerFactory
 {
     public static function createHandler(object|string $connection): AbstractSessionHandler
     {
+        if (\is_string($connection) && $options = parse_url($connection)) {
+            parse_str($options['query'] ?? '', $options);
+        } else {
+            $options = [];
+        }
+
         switch (true) {
             case $connection instanceof \Redis:
             case $connection instanceof \RedisArray:
@@ -54,7 +60,7 @@ public static function createHandler(object|string $connection): AbstractSession
                 $handlerClass = str_starts_with($connection, 'memcached:') ? MemcachedSessionHandler::class : RedisSessionHandler::class;
                 $connection = AbstractAdapter::createConnection($connection, ['lazy' => true]);
 
-                return new $handlerClass($connection);
+                return new $handlerClass($connection, array_intersect_key($options, ['prefix' => 1, 'ttl' => 1]));
 
             case str_starts_with($connection, 'pdo_oci://'):
                 if (!class_exists(DriverManager::class)) {
@@ -72,7 +78,7 @@ public static function createHandler(object|string $connection): AbstractSession
             case str_starts_with($connection, 'sqlsrv://'):
             case str_starts_with($connection, 'sqlite://'):
             case str_starts_with($connection, 'sqlite3://'):
-                return new PdoSessionHandler($connection);
+                return new PdoSessionHandler($connection, $options ?: []);
         }
 
         throw new \InvalidArgumentException(sprintf('Unsupported Connection: "%s".', $connection));