Development #5832
Améliorer la barre Mandaye quand un utilisateur est connecté localement
100%
Description
Lorsqu'on est connecté localement sur l'application mais pas sur l'idp il faudrait pouvoir adapter la bar Mandaye. L'idée c'est que Mandaye puisse appeler une fonction donné par le mapper afin que celle ci lui dise si le mec est connecté localement ou non.
Fichiers
Révisions associées
Historique
Mis à jour par Jérôme Schneider il y a plus de 9 ans
Pour tester le patch il faut suivre la "Quick installation" du README.rst
- rajouter 'linuxfrsaml.local 127.0.0.1' dans votre /etc/hosts.
- utiliser le fichier de configuration mandaye :
[mandaye] toolbar: true offline_toolbar: true
- modifier le PROJECT_NAME/mappers/linuxfr_example.py et rajouter par exemple ces lignes :
def test(env, request, response): return True #return False is_local_login = test
Se rendre sur http://linuxfrsaml.local:8000 pour voir tout ça.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Fichier 0001-test-if-the-mapper-have-a-is_local_login-arg-anf-mod.patch 0001-test-if-the-mapper-have-a-is_local_login-arg-anf-mod.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 9 ans
Il y a création d'un env['mandaye.auth'] qui n'est pas utilisé.
Pour l'utilisation de la fonction is_local_login, le code pourrait simplement être :
if hasattr(env['mandaye.mapper'], 'is_local_login'): values['is_local_login'] = env['mandaye.mapper'].is_local_login(env, request, response)
Par ailleurs je suggérerais un nom plus explicite à la fonction, is_user_locally_logged_in, par exemple. (il y a sans doute un tas d'autres trucs à renommer pour être plus lisible, mais il faut bien commencer quelque part).
Et ça peut aussi être le moment pour commencer une documentation sur ce que doit contenir un mapper (au-delà des lignes de commentaire en haut du linuxfr_example.py).
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Fichier 0001-add-option-is_user_locally_logged_in-to-mappers.patch 0001-add-option-is_user_locally_logged_in-to-mappers.patch ajouté
- Assigné à mis à Jérôme Schneider
Frédéric Péters a écrit :
Il y a création d'un env['mandaye.auth'] qui n'est pas utilisé.
Effectivement je n'en ai pas besoin ici c'était pour une autre utilisation. J'en ferais un patch séparait.
Pour l'utilisation de la fonction is_local_login, le code pourrait simplement être :
[...]
J'ai repris l'idée mais j'initialise tout de même la variable à None pour simplifier sa gestion dans les templates.
Par ailleurs je suggérerais un nom plus explicite à la fonction, is_user_locally_logged_in, par exemple. (il y a sans doute un tas d'autres trucs à renommer pour être plus lisible, mais il faut bien commencer quelque part).
J'ai renommé.
Et ça peut aussi être le moment pour commencer une documentation sur ce que doit contenir un mapper (au-delà des lignes de commentaire en haut du linuxfr_example.py).
Je viens d'ouvrir #5858. J'ai commencé mais je suis encore loin du compte.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit commit:58c6209c8f5ae54704459ddc6144b1b7dd7ef4ca.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Statut changé de Résolu (à déployer) à Fermé
add option is_user_locally_logged_in to mappers
Closes #5832