symfony
diff --git a/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategy.php
Lines changed: 16 additions & 8 deletions b/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategy.php
Lines changed: 16 additions & 8 deletions
diff --git a/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategyDecorator.php
Lines changed: 0 additions & 55 deletions b/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategyDecorator.php
Lines changed: 0 additions & 55 deletions
diff --git a/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/NotFoundActivationStrategyDecoratorTest.php
Lines changed: 0 additions & 56 deletions b/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/NotFoundActivationStrategyDecoratorTest.php
Lines changed: 0 additions & 56 deletions
diff --git a/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/NotFoundActivationStrategyTest.php
Lines changed: 18 additions & 4 deletions b/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/NotFoundActivationStrategyTest.php
Lines changed: 18 additions & 4 deletions
@@ -11,36 +11,44 @@
 
 namespace Symfony\Bridge\Monolog\Handler\FingersCrossed;
 
+use Monolog\Handler\FingersCrossed\ActivationStrategyInterface;
 use Monolog\Handler\FingersCrossed\ErrorLevelActivationStrategy;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 
-trigger_deprecation('symfony/monolog-bridge', '5.1', 'The "%s" class is deprecated, use "%s" instead.', NotFoundActivationStrategy::class, NotFoundActivationStrategyDecorator::class);
-
 /**
  * Activation strategy that ignores 404s for certain URLs.
  *
  * @author Jordi Boggiano <j.boggiano@seld.be>
  * @author Fabien Potencier <fabien@symfony.com>
- *
- * @deprecated since Symfony 5.1, Symfony\Bridge\Monolog\Handler\FingersCrossed\NotFoundActivationStrategyDecorator instead.
+ * @author Pierrick Vignand <pierrick.vignand@gmail.com>
  */
-class NotFoundActivationStrategy extends ErrorLevelActivationStrategy
+/* final */ class NotFoundActivationStrategy extends ErrorLevelActivationStrategy implements ActivationStrategyInterface
 {
+    private $inner;
     private $exclude;
     private $requestStack;
 
-    public function __construct(RequestStack $requestStack, array $excludedUrls, $actionLevel)
+    /**
+     * @param ActivationStrategyInterface|int|string $inner an ActivationStrategyInterface to decorate
+     */
+    public function __construct(RequestStack $requestStack, array $excludedUrls, $inner)
     {
-        parent::__construct($actionLevel);
+        if (!$inner instanceof ActivationStrategyInterface) {
+            trigger_deprecation('symfony/monolog-bridge', '5.2', 'Passing an actionLevel (int|string) as constructor\'s 3rd argument of "%s" is deprecated, "%s" expected.', __CLASS__, ActivationStrategyInterface::class);
+
+            $actionLevel = $inner;
+            $inner = new ErrorLevelActivationStrategy($actionLevel);
+        }
 
+        $this->inner = $inner;
         $this->requestStack = $requestStack;
         $this->exclude = '{('.implode('|', $excludedUrls).')}i';
     }
 
     public function isHandlerActivated(array $record): bool
     {
-        $isActivated = parent::isHandlerActivated($record);
+        $isActivated = $this->inner->isHandlerActivated($record);
 
         if (
             $isActivated
 
@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\Monolog\Tests\Handler\FingersCrossed;
 
+use Monolog\Handler\FingersCrossed\ErrorLevelActivationStrategy;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\Monolog\Handler\FingersCrossed\NotFoundActivationStrategy;
@@ -25,17 +26,30 @@ class NotFoundActivationStrategyTest extends TestCase
      *
      * @group legacy
      */
-    public function testIsActivated($url, $record, $expected)
+    public function testIsActivatedLegacy(string $url, array $record, bool $expected): void
     {
         $requestStack = new RequestStack();
         $requestStack->push(Request::create($url));
 
         $strategy = new NotFoundActivationStrategy($requestStack, ['^/foo', 'bar'], Logger::WARNING);
 
-        $this->assertEquals($expected, $strategy->isHandlerActivated($record));
+        self::assertEquals($expected, $strategy->isHandlerActivated($record));
     }
 
-    public function isActivatedProvider()
+    /**
+     * @dataProvider isActivatedProvider
+     */
+    public function testIsActivated(string $url, array $record, bool $expected): void
+    {
+        $requestStack = new RequestStack();
+        $requestStack->push(Request::create($url));
+
+        $strategy = new NotFoundActivationStrategy($requestStack, ['^/foo', 'bar'], new ErrorLevelActivationStrategy(Logger::WARNING));
+
+        self::assertEquals($expected, $strategy->isHandlerActivated($record));
+    }
+
+    public function isActivatedProvider(): array
     {
         return [
             ['/test',      ['level' => Logger::DEBUG], false],
@@ -50,7 +64,7 @@ public function isActivatedProvider()
         ];
     }
 
-    protected function getContextException($code)
+    protected function getContextException(int $code): array
     {
         return ['exception' => new HttpException($code)];
     }