Skip to content

Commit

Permalink
bug #780 Add deprecation message argument to JWTFactory.php (Chris Br…
Browse files Browse the repository at this point in the history
…adbury)

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

Discussion
----------

Add deprecation message argument to JWTFactory.php

Apologies if the message isn't quite right, but figured it was best to try a PR myself rather than expecting someone else to do it 🙂.

This should fix the remaining indirect deprecation notice due to the changes made to `setDeprecated()` in Symfony 5.1.

Closes #778

Commits
-------

99dd913 Add deprecation message argument to JWTFactory.php
  • Loading branch information
chalasr committed Oct 27, 2020
2 parents e434f3b + 99dd913 commit 3e541e8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion DependencyInjection/Security/Factory/JWTFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ public function addConfiguration(NodeDefinition $node)
{
$deprecationArgs = [];
if (method_exists(BaseNode::class, 'getDeprecation')) {
$deprecationArgs = ['lexik/jwt-authentication-bundle', '2.7'];
$deprecationArgs = ['lexik/jwt-authentication-bundle', '2.7', 'The "%path%.%node%" configuration key is deprecated. Use the "lexik_jwt_authentication.jwt_token_authenticator" Guard authenticator instead.'];
}

if (method_exists($node, 'setDeprecated')) {
Expand Down

0 comments on commit 3e541e8

Please sign in to comment.