bug #827 Use named constructor for lcobucci/jwt Ecdsa signers (chalasr)

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

Discussion
----------

Use named constructor for lcobucci/jwt Ecdsa signers

Fixes #823

Commits
-------

82b64a2 Use named constructor for lcobucci/jwt Ecdsa signers

Services/JWSProvider/LcobucciJWSProvider.php

@@ -217,6 +217,10 @@ private function getSignerForAlgorithm($signatureAlgorithm)
 
         $signerClass = $signerMap[$signatureAlgorithm];
 
+        if (is_subclass_of($signerClass, Signer\Ecdsa::class) && method_exists($signerClass, 'create')) {
+            return $signerClass::create();
+        }
+
         return new $signerClass();
     }
 

Tests/Services/JWSProvider/AbstractJWSProviderTest.php

@@ -181,7 +181,7 @@ public function testCreateWithExtraStandardClaims()
         $this->assertNotEmpty($created->getToken());
     }
 
-    private function getKeyLoaderMock()
+    protected function getKeyLoaderMock()
     {
         return $this
             ->getMockBuilder(static::$keyLoaderClass)

Tests/Services/JWSProvider/LcobucciJWSProviderTest.php

@@ -4,6 +4,7 @@
 
 use Lexik\Bundle\JWTAuthenticationBundle\Services\JWSProvider\LcobucciJWSProvider;
 use Lexik\Bundle\JWTAuthenticationBundle\Services\KeyLoader\RawKeyLoader;
+use Lexik\Bundle\JWTAuthenticationBundle\Signature\CreatedJWS;
 
 /**
  * Tests the LcobucciJWSProvider.
@@ -14,4 +15,25 @@ final class LcobucciJWSProviderTest extends AbstractJWSProviderTest
 {
     protected static $providerClass  = LcobucciJWSProvider::class;
     protected static $keyLoaderClass = RawKeyLoader::class;
+
+    public function testCreateWithEcdsa()
+    {
+        $keyLoaderMock = $this->getKeyLoaderMock();
+        $keyLoaderMock
+            ->expects($this->once())
+            ->method('loadKey')
+            ->with('private')
+            ->willReturn(self::$privateKey);
+        $keyLoaderMock
+            ->expects($this->once())
+            ->method('getPassphrase')
+            ->willReturn('foobar');
+
+        $payload     = ['username' => 'chalasr'];
+        $jwsProvider = new LcobucciJWSProvider($keyLoaderMock, 'openssl', 'EC512', 3600, 0);
+
+        $this->assertInstanceOf(CreatedJWS::class, $created = $jwsProvider->create($payload));
+
+        return $created->getToken();
+    }
 }

phpunit.xml.dist

@@ -26,4 +26,8 @@
             </exclude>
         </whitelist>
     </filter>
+
+    <php>
+        <env name="SYMFONY_DEPRECATIONS_HELPER" value="max[self]=0"></env>
+    </php>
 </phpunit>