Symfony 5 : Récupérer les données de l'utilisateur connecté

Version imprimable

Bonjour à tous,
J'essai d'utiliser la méthode getUser() pour récupérer les données de l'utilisateur connecté mais sans succès

doc :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public function index()
{
    // usually you'll want to make sure the user is authenticated first
    $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
 
    // returns your User object, or null if the user is not authenticated
    // use inline documentation to tell your editor your exact User class
    /** @var \App\Entity\User $user */
    $user = $this->getUser();
 
    // Call whatever methods you've added to your User class
    // For example, if you added a getFirstName() method, you can use that.
    return new Response('Well hi there '.$user->getFirstName());
}

Mon code

Code:

1
2
3
4
5
6
7
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $providerKey)
    {
        if ($targetPath = $this->getTargetPath($request->getSession(), $providerKey)) {
            return new RedirectResponse($targetPath);
        }
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
        $user = $this->getUser();

en ce qui concerne la ligne $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY'); j'ai une erreur

Citation:

"Undefined méthod denyAccessUnlessGranted"

mais l'appel de cette méthode est il vraiment nécessaire ?
Pour la ligne $user = $this->getUser(); j'ai un

Citation:

"Expected 2 arguments. Found 0"

24/01/2021, 21h26
_Thomas

Salut,

En fait la section de la doc à laquelle tu fais référence (https://symfony.com/doc/current/secu...he-user-object) indique comment récupérer l'utilisateur courant depuis un contrôleur en précisant que depuis un contrôleur, on peut accéder à des raccourcis :

Citation:

After authentication, the User object of the current user can be accessed via the getUser() shortcut

Or visiblement tu n'es pas dans un contrôleur et 'est pourquoi tu as les erreurs qui indiquent que les raccourcis vers ces méthodes n'existent pas. Pour t'en sortir, il faut suivre la section suivante (https://symfony.com/doc/current/secu...from-a-service) qui explique comment injecter le service Security de Symfony dans ton service (c'est à dire dans ta classe qui contient la méthode onAuthenticationSuccess).

Normalement en suivant la doc, tu devrais t'en sortir, sinon n'hésite pas à revenir ici !
25/01/2021, 15h03
LudovicKL

1 pièce(s) jointe(s)

Bonjour Thomas,
premièrement merci pour ta réponse,
Ta solution semble la bonne, en effet je n'ai plus le problème qui avait ouvert ce fil.
Par contre j'en ai un autre et qui est cette erreur :

Citation:

"App\Controller\RegistrationController::register(): Return value must be of type Symfony\Component\HttpFoundation\Response, null returned"

Pièce jointe 589295

Je me demande si je ne devrais pas ne pas modifier UserAuthenticator.php, déterminer uniquement la redirection vers un controller comme l'indique la doc et dans ce controller procéder à ma redirection finale. Cela m'ajoute un controller qui ne fait pas grand chose, mais cela m'évite d'introduire des conflits dans le code existant.
25/01/2021, 19h06
_Thomas

Citation:

Envoyé par LudovicKL

Je me demande si je ne devrais pas ne pas modifier UserAuthenticator.php, déterminer uniquement la redirection vers un controller comme l'indique la doc et dans ce controller procéder à ma redirection finale. Cela m'ajoute un controller qui ne fait pas grand chose, mais cela m'évite d'introduire des conflits dans le code existant.

Qu'est-ce que tu essayes de faire concrètement ? (n'hésite pas à partager le code en totalité et en utilisant les balises [CODE])

En gros suivant l'utilisateur, donc suivant son ROLE, il doit avoir accès à certaines fonctionnalités.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public function onAuthenticationSuccess(Request $request, TokenInterface $token, string $providerKey)
    {
        if ($targetPath = $this->getTargetPath($request->getSession(), $providerKey)) {
            return new RedirectResponse($targetPath);
        }
        $user = $this->security->getUser();
 
        if ($user->getRoles() == "ROLE_WORKER") {
            return new RedirectResponse($this->urlGenerator->generate('worker'));
        }
        //throw new \Exception('TODO: provide a valid redirect inside ' . __FILE__);
 
        if ($user->getRoles() == "ROLE_AGENCY") {
            return new RedirectResponse($this->urlGenerator->generate('agency'));
        }
    }

Donc j'ai essayé de le faire directement en modifiant la fonction onAuthenticationSucces(), je pense que c'est faisable, mais si je n'y arrive pas, je passerai par un controller intermédiaire comme expliqué précédemment.

26/01/2021, 18h54
_Thomas

Qu'est-ce qui ne fonctionne pas comme tu voudrais dans le code actuel ?
26/01/2021, 19h05
LudovicKL

J'ai l'erreur du #3
27/01/2021, 21h49
_Thomas

Donc si je comprends bien, dans la méthode register de ton contrôleur tu affiches un formulaire d'inscription, et quand l'utilisateur le valide tu voudrais créer l'utilisateur, l'authentifier immédiatement à partir des valeurs saisies dans le formulaire, et selon son rôle le rediriger vers telle ou telle URL. C'est bien ça ?

Pour faire ça tu as (entre autres) créé un GuardAuthenticator personnalisé, c'est bien ce Guard que tu passes en paramètre ligne 74. C'est toujours ça ?

A priori tout se passe bien, l'exécution atteint bien la méthode onAuthenticationSuccess de ton Guard, or cette méthode retourne probablement null, ce qui est autorisé (https://github.com/symfony/security-...rface.php#L136), mais ce qui fait que ton contrôleur retourne au final cette même valeur null, et là ce n'est plus autorisé, d'où l'erreur qui indique qu'il faut impérativement retourner une Response.

À mon avis le problème vient de ta méthode onAuthenticationSuccess : si tu ne rentres dans aucun "if", la méthode ne retourne jamais de Response. En l'occurrence, $user->getRoles() retourne un tableau de rôles et pas une simple string (https://github.com/symfony/symfony/b...erface.php#L47), donc il faudrait probablement plutôt des tests du type if(in_array('ROLE_WORKER', $user->getRoles())){...}.
30/01/2021, 17h33
LudovicKL
Je viens juste de voir ta réponse, désolé Thomas.
Effectivement, tu as raison, je viens de trouver mon erreur, getRoles() retourne un array, donc j'ai écrit
Code:

1 2 3 4 5 $roles = $user->getRoles() if ($roles[0] == 'ROLE_WORKER' { etc ...
mais ta solution est encore mieux.

Merci Thomas