search

PnX-SI / UsersHub-authentification-module

Module Flask d'authentification de UsersHub
GNU General Public License v3.0
5 stars 12 forks source link

Fix setup.py open() encoding #9

Closed ksamuel closed 6 years ago

ksamuel commented 6 years ago

Cette PR rajoute une gestion explicite de l'encoding du README dans le fichier setup.py.

On peut ainsi maintenant installer UsersHub-authentification-module avec pip:

pip install git+https://github.com/PnX-SI/UsersHub-authentification-module/

Et même l'ajouter dans un fichier requirements.txt.

ksamuel commented 6 years ago

Deuxième ajout, pas particulièrement lié au premier, mais qui s'est révélé pendant la mise en prod: le schéma de UsersHub n'utilise pas des clés primaires auto incrémentées, mais des séquences dont on prend l'élément suivant.

J'ai donc modifié le modèle pour refléter cela, ce qui évite des problèmes de clés dupliquées si on créés des utilisateurs de tests en local.

ksamuel commented 6 years ago

Troisième ajout, UsersHub possède au moins un utilisateur avec champ identifiant à "null". Dans ce cas str retourne une chaine vide, puisque que la méthode doit retourner une chaine.

amandine-sahl commented 6 years ago

Personnellement je ne sais pas pourquoi mais je n'avais aucun soucis lors de l'installation avec pip avant.

Concernant les séquences : La gestion des utilisateurs ne se fait pas via ce module mais via l'outil usersHub. Ce module ne doit théoriquement qu'apporter la possibilité de récupérer les droits de connexion d'un utilisateur dans des applications flasks.

J'accepte la pull request tout en considérant que la fonctionnalité insertion des utilisateurs ne doit être utilisée que pour des tests et qu'en production il faut par ailleur installer l'application UsersHub

camillemonchicourt commented 6 years ago

Oui et si on a besoin de la BDD uniquement il faut la prendre dans le depot de UsersHub et ne pas la dupliquer dans ce depot car on ne l'y maintient pqs.

Voir https://github.com/PnX-SI/UsersHub-authentification-module/issues/2