Discussion :

[LudovicKL]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

"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

"Expected 2 arguments. Found 0"

[_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 :

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 !

[LudovicKL]

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 :

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

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.

[_Thomas]

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])

[LudovicKL]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[_Thomas]

Qu'est-ce qui ne fonctionne pas comme tu voudrais dans le code actuel ?