symfony
diff --git a/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/HttpCodeActivationStrategy.php
Lines changed: 14 additions & 2 deletions b/‎src/Symfony/Bridge/Monolog/Handler/FingersCrossed/HttpCodeActivationStrategy.php
Lines changed: 14 additions & 2 deletions
diff --git a/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/HttpCodeActivationStrategyTest.php
Lines changed: 20 additions & 4 deletions b/‎src/Symfony/Bridge/Monolog/Tests/Handler/FingersCrossed/HttpCodeActivationStrategyTest.php
Lines changed: 20 additions & 4 deletions
@@ -25,8 +25,20 @@ class HttpCodeActivationStrategy extends ErrorLevelActivationStrategy
     private $exclusions;
     private $requestStack;
 
+    /**
+     * @param array $exclusions each exclusion must have a "code" and "urls" keys
+     */
     public function __construct(RequestStack $requestStack, array $exclusions, $actionLevel)
     {
+        foreach ($exclusions as $exclusion) {
+            if (!array_key_exists('code', $exclusion)) {
+                throw new \LogicException(sprintf('An exclusion must have a "code" key'));
+            }
+            if (!array_key_exists('urls', $exclusion)) {
+                throw new \LogicException(sprintf('An exclusion must have a "urls" key'));
+            }
+        }
+
         parent::__construct($actionLevel);
 
         $this->requestStack = $requestStack;
@@ -49,8 +61,8 @@ public function isHandlerActivated(array $record)
                 }
 
                 $urlBlacklist = null;
-                if (count($exclusion['url'])) {
-                    return !preg_match('{('.implode('|', $exclusion['url']).')}i', $request->getPathInfo());
+                if (count($exclusion['urls'])) {
+                    return !preg_match('{('.implode('|', $exclusion['urls']).')}i', $request->getPathInfo());
                 }
 
                 return false;
 
@@ -20,6 +20,22 @@
 
 class HttpCodeActivationStrategyTest extends TestCase
 {
+    /**
+     * @expectedException \LogicException
+     */
+    public function testExclusionsWithoutCode()
+    {
+        new HttpCodeActivationStrategy(new RequestStack(), array(array('urls' => array())), Logger::WARNING);
+    }
+
+    /**
+     * @expectedException \LogicException
+     */
+    public function testExclusionsWithoutUrls()
+    {
+        new HttpCodeActivationStrategy(new RequestStack(), array(array('code' => 404)), Logger::WARNING);
+    }
+
     /**
      * @dataProvider isActivatedProvider
      */
@@ -31,10 +47,10 @@ public function testIsActivated($url, $record, $expected)
         $strategy = new HttpCodeActivationStrategy(
             $requestStack,
             array(
-                array('code' => 403, 'url' => array()),
-                array('code' => 404, 'url' => array()),
-                array('code' => 405, 'url' => array()),
-                array('code' => 400, 'url' => array('^/400/a', '^/400/b')),
+                array('code' => 403, 'urls' => array()),
+                array('code' => 404, 'urls' => array()),
+                array('code' => 405, 'urls' => array()),
+                array('code' => 400, 'urls' => array('^/400/a', '^/400/b')),
             ),
             Logger::WARNING
         );