Merge pull request #22 from ConnectHolland/deprecate-success-handler

Update docs to use lexik default success handler
Harborn-digital · Nov 2, 2020 · bda1a15 · bda1a15
2 parents ece1155 + 5499134
commit bda1a15
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -62,6 +62,8 @@ lexik_jwt_authentication:
             cookie:
                 enabled: true
                 name:    BEARER
+                set_cookies:
+                    BEARER: ~
 ```
 
 ### Make sure the token is set as a secure cookie
@@ -74,7 +76,7 @@ In the `security.yaml` config file:
         anonymous: true
         json_login:
             check_path:               /api/login_check
-            success_handler:          ConnectHolland\SecureJWTBundle\Security\Http\Authentication\AuthenticationSuccessHandler
+            success_handler:          lexik_jwt_authentication.handler.authentication_success
             failure_handler:          lexik_jwt_authentication.handler.authentication_failure
 ```
 

diff --git a/src/Security/Http/Authentication/AuthenticationSuccessHandler.php b/src/Security/Http/Authentication/AuthenticationSuccessHandler.php
@@ -15,6 +15,9 @@
 use Symfony\Component\Security\Core\User\UserInterface;
 use Symfony\Component\Security\Http\Authentication\AuthenticationSuccessHandlerInterface;
 
+/**
+ * @deprecated Use lexik default instead
+ **/
 class AuthenticationSuccessHandler implements AuthenticationSuccessHandlerInterface
 {
     private AuthenticationSuccessHandlerInterface $successHandler;