Discussion :

[tsumiko]

Bonjour,

Je suis en train de mettre en place le login via sfGuard, et j'ai trouvé un truc bizarre pour ne pas dire étrange :
La saisie se fait via le mail et mot de passe, quand je fais un xdebug sur le php "sfGuardValidatorUser" dont voici une portion de code :

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
17
18
19
20
21
22
23
 
$allowEmail = sfConfig::get('app_sf_guard_plugin_allow_login_with_email', true);
    $method = $allowEmail ? 'retrieveByUsernameOrEmailAddress' : 'retrieveByUsername';
 
    // don't allow to sign in with an empty username
    if ($username)
    {
       if ($callable = sfConfig::get('app_sf_guard_plugin_retrieve_by_username_callable'))
       {
           $user = call_user_func_array($callable, array($username));
       } else {
           $user = $this->getTable()->retrieveByUsername($username);
       }
        // user exists?
       if($user)
       {
          // password is ok?
          if ($user->getIsActive() && $user->checkPassword($password))
          {
            return array_merge($values, array('user' => $user));
          }
       }
    }

la variable $method contient bien 'retrieveByUsernameOrEmailAddress' et lorsque j'arrive sur "if ($callable = sfConfig::get...", le programme va tester "retrieveByUsername" et non "retrieveByUsernameOrEmailAddress"
ce qui veut dire que la validation par mail ne fonctionne pas puisqu'il va faire un select via le pseudo et non via l'email.
Etrange non?
Comment l'obliger à faire le test via le mail sans remplacer retrieveByUsername" par "retrieveByUsernameOrEmailAddress" ou ajouter un test sur $methode avec un else, car c'est l'installation par defaut de sfGuard, et en règle générale il est absolument déconseiller de modifier les sources !

Si vous pouvez m'éclairer
Merci

[Michel Rotta]

J'espère que quelqu'un ici connait mieux sfGuard que moi

Ma réponse ne sera pas des plus encourageante. Je n'ai jamais utilisé que l'authentification par username. Et je n'ai jamais vu passer ici personne qui tente de faire l'un ou l'autre (ou même uniquement par email address).

Je ne suis pas sur que cela soit possible, même s'il reste dans le code un début de velléité de faire (ou, au contraire, une ouverture pour le future).

Je dirais que le mieux (mais lourd et beaucoup de conséquence) serait de modifier le plugin pour qu'il colle à tes besoins. Dans le lib local, si tu mets un objet du plugin, il sera pris en priorité par rapport a celui qui est dans plugin.

Je ne pense pas que ce plugin va beaucoup évoluer dans les années qui viennent, le risque est donc minime, avec la v2, c'est plutôt là que les modifications se porteront.

[dourouc05]

C'est une portion du plug-in qui m'a semblé fort obscure et ardue à l'utilisation. Je te conseillerais plus de repartir en effaçant toutes ces horreurs de ta tête et de mettre ta solution à la place de celle de sfGuard à ce niveau.

D'où cet article : http://www.developpez.net/forums/d99...ation-externe/. Tu peux alors surcharger ce que fait sfGuard au niveau de l'authentification ; même si c'est pour utiliser ce qu'il te propose mais un peu différemment, ça pourrait t'être utile.

La section où ce même code est montré et (un peu) décortiqué (pour les besoins de la suite, sans plus) : http://tcuvelier.developpez.com/php/...-externe/#LIII.

Ça sera probablement sortir l'arme nucléaire pour tuer la mouchette, mais, si ça fonctionne pour toi, c'est déjà ça de pris ! Au pire, modifie les sources du plug-in, si ça t'évite de faire un détour de quelques dizaines de ligne qui pourrait potentiellement mettre tout à terre (enfin, il m'a fallu bien chipoter pour arriver au résultat que je voulais).

la variable $method contient bien retrieveByUsernameOrEmailAddress' et lorsque j'arrive sur "if ($callable = sfConfig::get...", le programme va tester "retrieveByUsername" et non "retrieveByUsernameOrEmailAddress"

C'est malheureusement normal, la variable $method est déclarée, bien remplie... et abandonnée lâchement dans la nature, sans qu'on tente de la lire. Pour corriger ça, je propose de remplacer le premier bout de code par le second (à tester, évidemment, je ne l'ai pas fait).

Code :

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

1
2
3
       } else {
           $user = $this->getTable()->retrieveByUsername($username);
       }

Code :

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

1
2
3
4
5
6
} elseif($method == 'retrieveByUsernameOrEmailAddress') {
    if(!($user = $this->getTable()->retrieveByUsername($username)))
        $user = $this->getTable()->retrieveByEmailAddress($username);
    } else {
           $user = $this->getTable()->retrieveByUsername($username);
       }

Pas sûr du nom de la méthode.

[tsumiko]

Problème résolu

[Michel Rotta]

Mettre la solution apportera certainement une aide à ceux qui pourraient arriver ici avec la même question, lire ce sujet, et, pour l'instant, repartir frustrés heu non, pas cool

[tsumiko]

Voilà la solution sans toucher aux sources originaux:
Dans le fichier myUser.class.php, ajouter

Code :

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

1
2
3
4
5
 
    public static function checkUserByEmail($username)
    {
        return Doctrine::getTable('sfGuardUser')->retrieveByUsernameOrEmailAddress($username);
    }

Et dans l'app.yml :

Code :

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

1
2
3
 
  .sf_guard_plugin:
    sf_guard_plugin_retrieve_by_username_callable: [myUser,checkUserByEmail]