[FrameworkBundle][RateLimiter] deprecate RateLimiterFactory alias

kbond · kbond · commit 159e00cf93d1 · 2025-04-07T16:12:35.000-04:00
diff --git a/UPGRADE-7.3.md b/UPGRADE-7.3.md
@@ -59,6 +59,7 @@ FrameworkBundle
    because its default value will change in version 8.0
  * Deprecate the `--show-arguments` option of the `container:debug` command, as arguments are now always shown
  * Deprecate the `framework.validation.cache` config option
+ * Deprecate the `RateLimiterFactory` alias, use `RateLimiterFactoryInterface` instead
  * Deprecate setting the `framework.profiler.collect_serializer_data` config option to `false`
 
    When set to `true`, normalizers must be injected using the `NormalizerInterface`, and not using any concrete implementation.
diff --git a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md
@@ -23,6 +23,7 @@ CHANGELOG
    the `#[AsController]` attribute is no longer required
  * Deprecate setting the `framework.profiler.collect_serializer_data` config option to `false`
  * Set `framework.rate_limiter.limiters.*.lock_factory` to `auto` by default
+ * Deprecate `RateLimiterFactory` alias
 
 7.2
 ---
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -3266,10 +3266,11 @@ private function registerRateLimiterConfiguration(array $config, ContainerBuilde
             $limiterConfig['id'] = $name;
             $limiter->replaceArgument(0, $limiterConfig);
 
-            $container->registerAliasForArgument($limiterId, RateLimiterFactory::class, $name.'.limiter');
+            $factoryAlias = $container->registerAliasForArgument($limiterId, RateLimiterFactory::class, $name.'.limiter');
 
             if (interface_exists(RateLimiterFactoryInterface::class)) {
                 $container->registerAliasForArgument($limiterId, RateLimiterFactoryInterface::class, $name.'.limiter');
+                $factoryAlias->setDeprecated('symfony/dependency-injection', '7.3', 'The "%alias_id%" service alias is deprecated and will be removed in 8.0.').\sprintf(' You should use "%s %s.limiter" instead.', RateLimiterFactoryInterface::class, $name);
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -3266,10 +3266,11 @@ private function registerRateLimiterConfiguration(array $config, ContainerBuilde`
`3266`	`3266`	`$limiterConfig['id'] = $name;`
`3267`	`3267`	`$limiter->replaceArgument(0, $limiterConfig);`
`3268`	`3268`
`3269`		`- $container->registerAliasForArgument($limiterId, RateLimiterFactory::class, $name.'.limiter');`
	`3269`	`+ $factoryAlias = $container->registerAliasForArgument($limiterId, RateLimiterFactory::class, $name.'.limiter');`
`3270`	`3270`
`3271`	`3271`	`if (interface_exists(RateLimiterFactoryInterface::class)) {`
`3272`	`3272`	`$container->registerAliasForArgument($limiterId, RateLimiterFactoryInterface::class, $name.'.limiter');`
	`3273`	`+ $factoryAlias->setDeprecated('symfony/dependency-injection', '7.3', 'The "%alias_id%" service alias is deprecated and will be removed in 8.0.').\sprintf(' You should use "%s %s.limiter" instead.', RateLimiterFactoryInterface::class, $name);`
`3273`	`3274`	`}`
`3274`	`3275`	`}`
`3275`	`3276`	`}`