Improved error message when no supported user provider is found

wouterj · wouterj · commit a3406da4fa40 · 2020-01-26T17:39:13.000+01:00
diff --git a/src/Symfony/Component/Security/Core/User/ChainUserProvider.php b/src/Symfony/Component/Security/Core/User/ChainUserProvider.php
@@ -91,7 +91,7 @@ public function refreshUser(UserInterface $user)
             $e->setUsername($user->getUsername());
             throw $e;
         } else {
-            throw new UnsupportedUserException(sprintf('The account "%s" is not supported.', \get_class($user)));
+            throw new UnsupportedUserException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', \get_class($user)));
         }
     }
 
diff --git a/src/Symfony/Component/Security/Http/Firewall/ContextListener.php b/src/Symfony/Component/Security/Http/Firewall/ContextListener.php
@@ -292,7 +292,7 @@ protected function refreshUser(TokenInterface $token)
             return null;
         }
 
-        throw new \RuntimeException(sprintf('There is no user provider for user "%s".', $userClass));
+        throw new \RuntimeException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', $userClass));
     }
 
     private function safelyUnserialize(string $serializedToken)
diff --git a/src/Symfony/Component/Security/Http/RememberMe/AbstractRememberMeServices.php b/src/Symfony/Component/Security/Http/RememberMe/AbstractRememberMeServices.php
@@ -234,7 +234,7 @@ final protected function getUserProvider(string $class): UserProviderInterface
             }
         }
 
-        throw new UnsupportedUserException(sprintf('There is no user provider that supports class "%s".', $class));
+        throw new UnsupportedUserException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', $class));
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ public function refreshUser(UserInterface $user)`
`91`	`91`	`$e->setUsername($user->getUsername());`
`92`	`92`	`throw $e;`
`93`	`93`	`} else {`
`94`		`- throw new UnsupportedUserException(sprintf('The account "%s" is not supported.', \get_class($user)));`
	`94`	`+ throw new UnsupportedUserException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', \get_class($user)));`
`95`	`95`	`}`
`96`	`96`	`}`
`97`	`97`
Original file line number	Diff line number	Diff line change
`@@ -292,7 +292,7 @@ protected function refreshUser(TokenInterface $token)`
`292`	`292`	`return null;`
`293`	`293`	`}`
`294`	`294`
`295`		`- throw new \RuntimeException(sprintf('There is no user provider for user "%s".', $userClass));`
	`295`	`+ throw new \RuntimeException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', $userClass));`
`296`	`296`	`}`
`297`	`297`
`298`	`298`	`private function safelyUnserialize(string $serializedToken)`
Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,7 @@ final protected function getUserProvider(string $class): UserProviderInterface`
`234`	`234`	`}`
`235`	`235`	`}`
`236`	`236`
`237`		`- throw new UnsupportedUserException(sprintf('There is no user provider that supports class "%s".', $class));`
	`237`	`+ throw new UnsupportedUserException(sprintf('There is no user provider for user "%s". Shouldn\'t the "supportsClass()" method of your user provider return true for this classname?', $class));`
`238`	`238`	`}`
`239`	`239`
`240`	`240`	`/**`