Bug #9415
MandayeJS : s'assurer de l'authentification lors de l'association
0%
Fichiers
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-association-failure-handled-9415.patch 0001-association-failure-handled-9415.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Josué Kouka il y a plus de 8 ans
Mis à jour par Frédéric Péters il y a plus de 8 ans
Il faut laisser la modif au .po en-dehors du commit.
Le json d'échange contient désormais url et uri, il devient utile de documenter un peu les différents attributs échangés.
Dans is_account_linked, ça existait déjà mais comme le nombre d'instructions augmente, il faut vraiment éviter le except: nu, qui ne précise pasa ce qu'il attrape.
Plus loin, dans disassociate le except: était précis, et là ne l'est plus.
Le point important pour finir : j'avais suggéré que l'expression d'évaluation de succès (/ d'échec) soit paramétrable (comme locators), ici c'est un systématique uri !== input.homepath, il ne me semble pas que ça tienne la route. (pas bien difficile d'imaginer un site qui n'envoie pas vers la racine après une authent à succès) (ni une authent qui a lieu à la racine et dont l'échec reste sur la racine).
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-association-failure-handled-9415.patch 0001-association-failure-handled-9415.patch ajouté
Le point important pour finir : j'avais suggéré que l'expression d'évaluation de succès (/ d'échec) soit paramétrable (comme locators), ici c'est un systématique uri !== input.homepath, il ne me semble pas que ça tienne la route. (pas bien difficile d'imaginer un site qui n'envoie pas vers la racine après une authent à succès) (ni une authent qui a lieu à la racine et dont l'échec reste sur la racine).
Nouvel attribut dans le settings.
SITE_AUTH_CHECKER
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-association-failure-handled-readme-updated-9415.patch 0001-association-failure-handled-readme-updated-9415.patch ajouté
readme updated
Mis à jour par Frédéric Péters il y a plus de 8 ans
Ok; reste encore à séparer la modification au .po.
Et le is_linked(), tu peux le retirer, il me semble.
Mis à jour par Josué Kouka il y a plus de 8 ans
Mis à jour par Frédéric Péters il y a plus de 8 ans
Pour moi c'est ok; je changerais juste le sujet du commit en "handle association failures (#9415)" ("Describe your changes in imperative mood", http://git.kernel.org/cgit/git/git.git/tree/Documentation/SubmittingPatches).
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Fermé
handle association failure (#9415)