Merge pull request #32 from rudott/master

Added checkPreAuth call before checkPostAuth
lightSAML · Nov 4, 2016 · 25b4f3c · 25b4f3c
2 parents c4b1765 + e751bfe
commit 25b4f3c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/...ightSaml/SpBundle/Security/Authentication/Provider/LightsSamlSpAuthenticationProvider.php b/...ightSaml/SpBundle/Security/Authentication/Provider/LightsSamlSpAuthenticationProvider.php
@@ -119,6 +119,7 @@ public function authenticate(TokenInterface $token)
         }
 
         if ($this->userChecker && $user instanceof UserInterface) {
+            $this->userChecker->checkPreAuth($user);
             $this->userChecker->checkPostAuth($user);
         }
 

diff --git a/...pBundle/Tests/Security/Authentication/Provider/LightsSamlSpAuthenticationProviderTest.php b/...pBundle/Tests/Security/Authentication/Provider/LightsSamlSpAuthenticationProviderTest.php
@@ -284,6 +284,10 @@ public function test_calls_user_checker_if_provided()
             ->method('loadUserByUsername')
             ->willReturn($user);
 
+        $userCheckerMock->expects($this->once())
+            ->method('checkPreAuth')
+            ->with($user);
+
         $userCheckerMock->expects($this->once())
             ->method('checkPostAuth')
             ->with($user);