[Security] Minor fixes

wouterj · wouterj · commit 7e213e312c1d · 2021-08-17T17:47:21.000+02:00
diff --git a/Authenticator/AbstractPreAuthenticatedAuthenticator.php b/Authenticator/AbstractPreAuthenticatedAuthenticator.php
@@ -113,7 +113,7 @@ public function createAuthenticatedToken(PassportInterface $passport, string $fi
 
     public function createToken(Passport $passport, string $firewallName): TokenInterface
     {
-        return new PreAuthenticatedToken($passport->getUser(), null, $firewallName, $passport->getUser()->getRoles());
+        return new PreAuthenticatedToken($passport->getUser(), $firewallName, $passport->getUser()->getRoles());
     }
 
     public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response
diff --git a/Authenticator/HttpBasicAuthenticator.php b/Authenticator/HttpBasicAuthenticator.php
@@ -95,7 +95,7 @@ public function createAuthenticatedToken(PassportInterface $passport, string $fi
 
     public function createToken(Passport $passport, string $firewallName): TokenInterface
     {
-        return new UsernamePasswordToken($passport->getUser(), null, $firewallName, $passport->getUser()->getRoles());
+        return new UsernamePasswordToken($passport->getUser(), $firewallName, $passport->getUser()->getRoles());
     }
 
     public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ public function createAuthenticatedToken(PassportInterface $passport, string $fi`
`113`	`113`
`114`	`114`	`public function createToken(Passport $passport, string $firewallName): TokenInterface`
`115`	`115`	`{`
`116`		`- return new PreAuthenticatedToken($passport->getUser(), null, $firewallName, $passport->getUser()->getRoles());`
	`116`	`+ return new PreAuthenticatedToken($passport->getUser(), $firewallName, $passport->getUser()->getRoles());`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ public function createAuthenticatedToken(PassportInterface $passport, string $fi`
`95`	`95`
`96`	`96`	`public function createToken(Passport $passport, string $firewallName): TokenInterface`
`97`	`97`	`{`
`98`		`- return new UsernamePasswordToken($passport->getUser(), null, $firewallName, $passport->getUser()->getRoles());`
	`98`	`+ return new UsernamePasswordToken($passport->getUser(), $firewallName, $passport->getUser()->getRoles());`
`99`	`99`	`}`
`100`	`100`
`101`	`101`	`public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $firewallName): ?Response`