bug #542 Fix missing implements breaking JWT header alteration (tucks…

…aun)

This PR was merged into the 2.x-dev branch.

Discussion
----------

Fix missing implements breaking JWT header alteration

Otherwise headers are not get from the event, see  https://github.com/lexik/LexikJWTAuthenticationBundle/blob/85b05784fbad7d6435626915edeeb55ed0bcd6a7/Services/JWTManager.php#L61-L65

Commits
-------

53755d0 Fix missing implements breaking JWT header alteration

Encoder/LcobucciJWTEncoder.php

@@ -11,7 +11,7 @@
  *
  * @author Robin Chalas <robin.chalas@gmail.com>
  */
-class LcobucciJWTEncoder implements JWTEncoderInterface
+class LcobucciJWTEncoder implements JWTEncoderInterface, HeaderAwareJWTEncoderInterface
 {
     /**
      * @var JWSProviderInterface

Tests/Functional/GetTokenTest.php

@@ -2,6 +2,7 @@
 
 namespace Lexik\Bundle\JWTAuthenticationBundle\Tests\Functional;
 
+use Lcobucci\JWT\Parser;
 use Lexik\Bundle\JWTAuthenticationBundle\Event\JWTCreatedEvent;
 use Lexik\Bundle\JWTAuthenticationBundle\Events;
 use Lexik\Bundle\JWTAuthenticationBundle\Response\JWTAuthenticationSuccessResponse;
@@ -41,6 +42,9 @@ public function testGetTokenWithCustomClaim()
 
         $this->assertArrayHasKey('custom', $payload, 'The payload should contains a "custom" claim.');
         $this->assertSame('dummy', $payload['custom'], 'The "custom" claim should be equal to "dummy".');
+
+        $jws = (new Parser())->parse((string) $body['token']);
+        $this->assertArrayHasKey('foo', $jws->getHeaders(), 'The payload should contains a custom "foo" header.');
     }
 
     public function testGetTokenFromInvalidCredentials()