Depreciate injection of Request instances

Event/AuthenticationFailureEvent.php

@@ -17,6 +17,8 @@ class AuthenticationFailureEvent extends Event
 {
     /**
      * @var Request
+     *
+     * @deprecated since 1.7, removed in 2.0
      */
     protected $request;
 
@@ -31,22 +33,33 @@ class AuthenticationFailureEvent extends Event
     protected $response;
 
     /**
-     * @param Request                 $request
+     * @param Request|null            $request   Deprecated
      * @param AuthenticationException $exception
      * @param Response                $response
      */
-    public function __construct(Request $request, AuthenticationException $exception, Response $response)
+    public function __construct(Request $request = null, AuthenticationException $exception, Response $response)
     {
-        $this->request   = $request;
+        if (null !== $request && class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Passing a Request instance as first argument of %s() is deprecated since version 1.7 and will be removed in 2.0.%sInject the "@request_stack" service in your event listener instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+
+            $this->request = $request;
+        }
+
         $this->exception = $exception;
         $this->response  = $response;
     }
 
     /**
+     * @deprecated since 1.7, removed in 2.0
+     *
      * @return Request
      */
     public function getRequest()
     {
+        if (class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Method %s() is deprecated since version 1.7 and will be removed in 2.0.%sUse  Symfony\Component\HttpFoundation\RequestStack::getCurrentRequest() instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+        }
+
         return $this->request;
     }
 

Event/AuthenticationSuccessEvent.php

@@ -27,6 +27,8 @@ class AuthenticationSuccessEvent extends Event
 
     /**
      * @var Request
+     *
+     * @deprecated since 1.7, removed in 2.0
      */
     protected $request;
 
@@ -38,11 +40,17 @@ class AuthenticationSuccessEvent extends Event
     /**
      * @param array         $data
      * @param UserInterface $user
-     * @param Request       $request
+     * @param Request|null  $request Deprecated
      * @param Response      $response
      */
-    public function __construct(array $data, UserInterface $user, Request $request, Response $response)
+    public function __construct(array $data, UserInterface $user, Request $request = null, Response $response)
     {
+        if (null !== $request && class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Passing a Request instance as first argument of %s() is deprecated since version 1.7 and will be removed in 2.0.%sInject the "@request_stack" service in your event listener instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+
+            $this->request = $request;
+        }
+
         $this->data = $data;
         $this->user = $user;
         $this->request = $request;
@@ -74,10 +82,16 @@ public function getUser()
     }
 
     /**
+     * @deprecated since 1.7, removed in 2.0
+     *
      * @return Request
      */
     public function getRequest()
     {
+        if (class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Method %s() is deprecated since version 1.7 and will be removed in 2.0.%sUse  Symfony\Component\HttpFoundation\RequestStack::getCurrentRequest() instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+        }
+
         return $this->request;
     }
 

Event/JWTAuthenticatedEvent.php

@@ -21,12 +21,7 @@ class JWTAuthenticatedEvent extends Event
      * @var TokenInterface
      */
     protected $token;
-
-    /**
-     * @var Request
-     */
-    protected $request;
-
+
     /**
      * @param array          $payload
      * @param TokenInterface $token

Event/JWTCreatedEvent.php

@@ -24,19 +24,26 @@ class JWTCreatedEvent extends Event
 
     /**
      * @var Request
+     *
+     * @deprecated since 1.7, removed in 2.0
      */
     protected $request;
 
     /**
      * @param array         $data
      * @param UserInterface $user
-     * @param Request       $request
+     * @param Request|null  $request Deprecated
      */
     public function __construct(array $data, UserInterface $user, Request $request = null)
     {
-        $this->data    = $data;
-        $this->user    = $user;
-        $this->request = $request;
+        if (null !== $request && class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Passing a Request instance as first argument of %s() is deprecated since version 1.7 and will be removed in 2.0.%sInject the "@request_stack" service in your event listener instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+
+            $this->request = $request;
+        }
+
+        $this->data = $data;
+        $this->user = $user;
     }
 
     /**
@@ -64,10 +71,16 @@ public function getUser()
     }
 
     /**
+     * @deprecated since 1.7, removed in 2.0
+     *
      * @return Request
      */
     public function getRequest()
     {
+        if (class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Method %s() is deprecated since version 1.7 and will be removed in 2.0.%sUse  Symfony\Component\HttpFoundation\RequestStack::getCurrentRequest() instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+        }
+
         return $this->request;
     }
 }

Event/JWTDecodedEvent.php

@@ -19,6 +19,8 @@ class JWTDecodedEvent extends Event
 
     /**
      * @var Request
+     *
+     * @deprecated since 1.7, removed in 2.0
      */
     protected $request;
 
@@ -28,13 +30,18 @@ class JWTDecodedEvent extends Event
     protected $isValid;
 
     /**
-     * @param array   $payload
-     * @param Request $request
+     * @param array        $payload
+     * @param Request|null $request Deprecated
      */
     public function __construct(array $payload, Request $request = null)
     {
+        if (null !== $request && class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Passing a Request instance as first argument of %s() is deprecated since version 1.7 and will be removed in 2.0.%sInject the "@request_stack" service in your event listener instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+
+            $this->request = $request;
+        }
+
         $this->payload = $payload;
-        $this->request = $request;
         $this->isValid = true;
     }
 
@@ -47,10 +54,16 @@ public function getPayload()
     }
 
     /**
+     * @deprecated since 1.7, removed in 2.0
+     *
      * @return Request
      */
     public function getRequest()
     {
+        if (class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Method %s() is deprecated since version 1.7 and will be removed in 2.0.%sUse  Symfony\Component\HttpFoundation\RequestStack::getCurrentRequest() instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+        }
+
         return $this->request;
     }
 

Event/JWTFailureEventInterface.php

@@ -8,7 +8,7 @@
 
 /**
  * Interface for event classes that are dispatched when a JWT cannot be authenticated.
- * 
+ *
  * @author Robin Chalas <[email protected]>
  */
 interface JWTFailureEventInterface
@@ -19,6 +19,8 @@ interface JWTFailureEventInterface
     public function getResponse();
 
     /**
+     * @deprecated since 1.7, removed in 2.0
+     *
      * @return Request
      */
     public function getRequest();

Event/JWTNotFoundEvent.php

@@ -7,21 +7,26 @@
 use Symfony\Component\Security\Core\Exception\AuthenticationException;
 
 /**
- * JWTNotFoundEvent event is dispatched when a JWT cannot be found in a request 
+ * JWTNotFoundEvent event is dispatched when a JWT cannot be found in a request
  * covered by a firewall secured via lexik_jwt.
  *
  * @author Robin Chalas <[email protected]>
  */
 class JWTNotFoundEvent extends AuthenticationFailureEvent implements JWTFailureEventInterface
 {
     /**
-     * @param Request                      $request
+     * @param Request|null                 $request   Deprecated
      * @param AuthenticationException|null $exception
      * @param Response|null                $response
      */
-    public function __construct(Request $request, AuthenticationException $exception = null, Response $response = null)
+    public function __construct(Request $request = null, AuthenticationException $exception = null, Response $response = null)
     {
-        $this->request   = $request;
+        if (null !== $request && class_exists('Symfony\Component\HttpFoundation\RequestStack')) {
+            @trigger_error(sprintf('Passing a Request instance as first argument of %s() is deprecated since version 1.7 and will be removed in 2.0.%sInject the "@request_stack" service in your event listener instead.', __METHOD__, PHP_EOL), E_USER_DEPRECATED);
+
+            $this->request = $request;
+        }
+
         $this->exception = $exception;
         $this->response  = $response;
     }

LexikJWTAuthenticationBundle.php

@@ -25,7 +25,5 @@ public function build(ContainerBuilder $container)
         /** @var SecurityExtension $extension */
         $extension = $container->getExtension('security');
         $extension->addSecurityListenerFactory(new JWTFactory());
-
-        $container->addCompilerPass(new RequestCompilerPass());
     }
 }

Resources/doc/2-data-customization.md

@@ -13,6 +13,7 @@ but you can add your own data.
 services:
     acme_api.event.jwt_created_listener:
         class: Acme\Bundle\ApiBundle\EventListener\JWTCreatedListener
+        arguments: [ '@request_stack' ] # Symfony 2.4+
         tags:
             - { name: kernel.event_listener, event: lexik_jwt_authentication.on_jwt_created, method: onJWTCreated }
 ```
@@ -21,18 +22,38 @@ Example 1 : add client ip to the encoded payload
 
 ``` php
 // Acme\Bundle\ApiBundle\EventListener\JWTCreatedListener.php
+
+use Symfony\Component\HttpFoundation\RequestStack;
+
 class JWTCreatedListener
 {
+    /**
+     * @var RequestStack
+     */
+    private $requestStack;
+
+    /**
+     * @param RequestStack $requestStack
+     */
+    public function __construct(RequestStack $requestStack)
+    {
+        $this->requestStack = $requestStack;
+    }
+
+    // ...
+
     /**
      * @param JWTCreatedEvent $event
      *
      * @return void
      */
     public function onJWTCreated(JWTCreatedEvent $event)
     {
-        if (!($request = $event->getRequest())) {
-            return;
-        }
+        // Symfony < 2.4
+        $request = $event->getRequest();
+
+        // Symfony 2.4+
+        $request = $this->requestStack->getCurrentRequest();
 
         $payload       = $event->getData();
         $payload['ip'] = $request->getClientIp();
@@ -47,7 +68,7 @@ Example 2 : override token expiration date calcul to be more flexikble
 ``` php
 // Acme\Bundle\ApiBundle\EventListener\JWTCreatedListener.php
 class JWTCreatedListener
-{
+{  
     /**
      * @param JWTCreatedEvent $event
      *
@@ -75,6 +96,7 @@ You can access the jwt payload once it has been decoded to perform you own addit
 services:
     acme_api.event.jwt_decoded_listener:
         class: Acme\Bundle\ApiBundle\EventListener\JWTDecodedListener
+        arguments: [ '@request_stack' ] # Symfony 2.4+
         tags:
             - { name: kernel.event_listener, event: lexik_jwt_authentication.on_jwt_decoded, method: onJWTDecoded }
 ```
@@ -85,19 +107,22 @@ Example 3 : check client ip the decoded payload (from example 1)
 // Acme\Bundle\ApiBundle\EventListener\JWTDecodedListener.php
 class JWTDecodedListener
 {
+    // ...
+
     /**
      * @param JWTDecodedEvent $event
      *
      * @return void
      */
     public function onJWTDecoded(JWTDecodedEvent $event)
     {
-        if (!($request = $event->getRequest())) {
-            return;
-        }
-
-        $payload = $event->getPayload();
+        // Symfony < 2.4
         $request = $event->getRequest();
+
+        // Symfony 2.4+
+        $request = $this->requestStack->getCurrentRequest();
+
+        $payload = $event->getPayload();
 
         if (!isset($payload['ip']) || $payload['ip'] !== $request->getClientIp()) {
             $event->markAsInvalid();
@@ -304,4 +329,4 @@ public function onJWTNotFound(JWTNotFoundEvent $event)
 
 __Protip:__ You might want to use the same method for customizing the response on both `JWT_INVALID` and `JWT_NOT_FOUND` events. 
 For that, use the `Event\JWTFailureEventInterface` interface to typehint the event argument of your listener's method, rather than 
-a specific event class (i.e. `JWTNotFoundEvent` or `JWTInvalidEvent`).
+a specific event class (i.e. `JWTNotFoundEvent` or `JWTInvalidEvent`).