Working on usershub-auth-module genericity

ksamuel commented 7 years ago

Bonjour,

Pour le projet du Mercantour j'avais besoin de certaines fonctionalités:

En développant celles-ci, j'ai aussi noté:

que les mots de passe de usershub étaient hashés en md5
que les tables liées aux droits étaient nécessaires pour certaines opérations
j'avais introduit une régression dans ma dernière PR
on charge le modèle user plusieurs fois après le login
la gestion des exceptions et codes de retour a besoin d'un peu de nettoyage
le cookie de session est renouvelé dans certains contextes où on ne le souhaite pas
les modèles n'ont pas de __repr__
AppUser affiche une vue en lecture seule et on ne peut donc pas setter le password

Voici donc une PR qui vise ces problématiques. Points clés :

Models:

nouvelles classes pour mapper les tables bib_droits et cor_role_droit_application
la classe User utilise md5 pour hasher son password au lieu de sha256. Il faudrait alerter UsersHub de changer le hashing de leur password pour un algo plus robuste et avec un salt. Mais en attendant on doit utiliser le leur.
- __repr__ pour faciliter le debuggage
- AppUser n'a plus de setter sur le password puisque c'est une vue en lecture seule

Auth workflow:

exceptions plus granulaires et exceptions personnalisées
obtenir un objet User depuis un token est maintenant une fonction indépendante
ajout d'une vue pour le log out
pas de renouvellement de cookie si le token est vide ou en cours d'écriture
redirection optionnelle sur check_auth
usage optionnel des codes HTTP standards pour les erreurs
le modèle user est maintenant attaché à Flask.g
COOKIE_AUTORENEW passe sur True par défaut pour éviter d'avoir à setter la valeur pour les projets existant. Une erreur de ma part dans la première PR.

camillemonchicourt commented 7 years ago

Super boulot mais il aurait fallu faire relire par @amandine-sahl avant de merger.

ksamuel commented 7 years ago

+1 je serais moi-même plus à l'aise si amandine validait les changements.

Je propose de créer une PR inverse, l'appliquer. Puis une nouvelle PR inverse en attente de validation par amandine.

Je n'ai pas les droits sur ce repo donc je ne peux pas faire cette procédure moi-même, mais voici les étapes: https://help.github.com/articles/reverting-a-pull-request/

samuelpriou commented 7 years ago

Désolé Amandine, vas y pour la PR inverse.

camillemonchicourt commented 7 years ago

PnX-SI / UsersHub-authentification-module