Merge branch '6.3' into 6.4

* 6.3:
  [Cache] Fix Redis6Proxy
  [Finder] Disable failing test about open_basedir
  Fix merge
  Fix merge
  [Notifier] Telegram Bridge add escaping for \
  [Routing] Fix routing collection defaults when adding a new route to a collection
  [Messenger] Fix cloned TraceableStack not unstacking the stack independently
  [DependencyInjection] Fix autocasting null env values to empty string with container.env_var_processors_locator
  [Cache] Fix support for Redis Sentinel using php-redis 6.0.0
  [Notifier] Fix Smsmode HttpClient mandatory headers
  minor #51693 Disable the dead code analysis in Psalm (stof)
  Update the PR template
  [SecurityBundle][PasswordHasher] Fix password migration with custom hasher service with security bundle config
  [Translator] Fix support for `default_path` in XML
  [FrameworkBundle] Handle tags array attributes in descriptors
  [HttpClient] Fix TraceableResponse if response has no destruct method
  [FrameworkBundle] Always use buildDir as `ConfigBuilderGenerator` outputDir

Author: nicolas-grekas

Loader/Configurator/CollectionConfigurator.php

@@ -117,4 +117,12 @@ final public function host(string|array $host): static
 
         return $this;
     }
+
+    /**
+     * This method overrides the one from LocalizedRouteTrait.
+     */
+    private function createRoute(string $path): Route
+    {
+        return (clone $this->route)->setPath($path);
+    }
 }

Tests/Fixtures/collection-defaults.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Symfony\Component\Routing\Loader\Configurator;
+
+return function (RoutingConfigurator $routes) {
+    $collection = $routes->collection();
+    $collection
+        ->methods(['GET'])
+        ->defaults(['attribute' => true])
+        ->stateless();
+
+    $collection->add('defaultsA', '/defaultsA')
+        ->locale('en')
+        ->format('html');
+
+    $collection->add('defaultsB', '/defaultsB')
+        ->methods(['POST'])
+        ->stateless(false)
+        ->locale('en')
+        ->format('html');
+};

Tests/Loader/PhpFileLoaderTest.php

@@ -103,6 +103,31 @@ public function testLoadingRouteWithDefaults()
         $this->assertSame('html', $defaultsRoute->getDefault('_format'));
     }
 
+    public function testLoadingRouteWithCollectionDefaults()
+    {
+        $loader = new PhpFileLoader(new FileLocator([__DIR__.'/../Fixtures']));
+        $routes = $loader->load('collection-defaults.php');
+
+        $this->assertCount(2, $routes);
+
+        $defaultsRoute = $routes->get('defaultsA');
+        $this->assertSame(['GET'], $defaultsRoute->getMethods());
+        $this->assertArrayHasKey('attribute', $defaultsRoute->getDefaults());
+        $this->assertTrue($defaultsRoute->getDefault('_stateless'));
+        $this->assertSame('/defaultsA', $defaultsRoute->getPath());
+        $this->assertSame('en', $defaultsRoute->getDefault('_locale'));
+        $this->assertSame('html', $defaultsRoute->getDefault('_format'));
+
+        // The second route has a specific method and is not stateless, overwriting the collection settings
+        $defaultsRoute = $routes->get('defaultsB');
+        $this->assertSame(['POST'], $defaultsRoute->getMethods());
+        $this->assertArrayHasKey('attribute', $defaultsRoute->getDefaults());
+        $this->assertFalse($defaultsRoute->getDefault('_stateless'));
+        $this->assertSame('/defaultsB', $defaultsRoute->getPath());
+        $this->assertSame('en', $defaultsRoute->getDefault('_locale'));
+        $this->assertSame('html', $defaultsRoute->getDefault('_format'));
+    }
+
     public function testLoadingImportedRoutesWithDefaults()
     {
         $loader = new PhpFileLoader(new FileLocator([__DIR__.'/../Fixtures']));