Re: [sympa-fr] Auth.pm, CAS et légers bugs

[Olivier Salaün - CRU <adresse@cachée>]

Bonjour Fabrice,

Fabrice Jammes wrote:

Il y a un très léger bug dans le fichier auth.pm de sympa 5.1
(il est toujours dans la 5.2 à la ligne 337)

On doit mettre :
$attributes->{'uid'} et non: $attribute->{'uid'}
[...]
   # fjammes 20060101 : il manquait le 's' a attributes
   do_log ('info',"Auth::get_email_by_net_id($auth_id,$attributes->{'uid'})");

Ok, on a corrigé cet appel à do_log().

Sinon, un autre petit détail qui m'a fait perdre pas mal de temps:

dans le auth.conf, directive ldap_host

si l'on met une liste de serveurs LDAP avec des espaces au niveau
des virgules alors il n'arrive pas à récupérer ces serveurs :

En effet dans auth.conf :

ldap_host    ldap1.univ.fr,ldap2.univ.fr,ldap2.univ.fr
fonctionne mais
ldap_host    ldap1.univ.fr, ldap2.univ.fr, ldap2.univ.fr

ne fonctionne pas, ce qui est assez déconcertant (de plus aucun message d'erreur ne s'affiche dans les logs).

Effectivement, on a un problème de remontée des erreurs lors du chargement des fichiers de config principaux. A améliorer...

Cela vient peut-être de l'_expression_ régulière suivante sur ldap_host (dans Conf.pm) :

             'cas' => {'base_url' => 'http(s)?:/.*',
                   'non_blocking_redirection' => 'on|off',
                   'login_path' => '.*',
                   'logout_path' => '.*',
                   'service_validate_path' => '.*',
                   'proxy_path' => '.*',
                   'proxy_validate_path' => '.*',
                   'auth_service_name' => '.*',
                   'authentication_info_url' => 'http(s)?:/.*',
                   'ldap_host' => '[\w\.\-]+(:\d+)?(,[\w\.\-]+(:\d+)?)*',

Les espaces ne sont pas parsés dans l'_expression_ régulière (pour elle fonctionne avec egrep, alors le problème ne vient peut-être pas de là), pourrait-on régler ce problème, cela ferait certainement gagner du temps à d'autres personnes qui souhaitent cassifier sympa ? Il faudrait également faire un trim sur les noms des serveurs LDAP lors du parsage de la chaîne ldap_host.

On a ajouté cela dans la branche de développement : http://sourcesup.cru.fr/cgi/viewcvs.cgi/sympa/src/Conf.pm?r1=1.123&r2=1.124

De plus, après authentification magique on a le message *"you_have_been_authenticated"
*qui s'affiche, on peut le traduire en français ?

On a déja amélioré ce message (et d'autres) dans la version de développement. Si tu veux effectuer une correction immédiate sur ton service de listes, il te faut dupliquer le template web 'notice.tt2' et ajouter la condition suivante :

[% ELSIF notice.msg == 'you_have_been_authenticated' %]
Vous avez été authentifié.

Merci ne nous avoir remonté toutes ces remarques.