Development #8826
Refactoring: merge LibertyServiceProvider into LibertyProvider
0%
Description
There is no need for two models anymore, all fields from LibertyServiceProvider could be carried by LibertyProvider.
- First a migration should add new field to LibertyProvider mapping those of LibertyServiceProvider with null=True for those lacking it
- Second a migration should copy all value from LibertyServiceProvider to their liberty_provider
- Third all use of LibertyServiceProvider should be changed to use directly LibertyProvider
- Finally the LibertyServiceProvider model should be removed
Fichiers
Demandes liées
Historique
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier liberty_service_provider-to-service_provider-8826.patch liberty_service_provider-to-service_provider-8826.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Je n'arrive plus à appliquer le patch sur master, pourrais-tu le rebaser ?
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch ajouté
Pour te remettre dans le contexte, l'erreur survenait dans src/authentic2/saml/models.py ligne 34. lasso.Provider
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Patch to remove last traces of handling of LibertyServiceProvider in authentic2/saml/forms.py, the form should be refactored as a ModelForm to remove the need to implement def save()
.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Fichier 0001-authentic2-saml-forms-remove-self.childs-there-are-n.patch 0001-authentic2-saml-forms-remove-self.childs-there-are-n.patch ajouté
Patch to merge with yours.
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch ajouté
LibertyFederation data migration added
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch ajouté
References to LibertyServiceProvider removed from common.py
Mis à jour par Josué Kouka il y a plus de 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch ajouté
fix FieldError: Relation fields do not support nested lookups
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
authentic2/saml/admin.py
remove commented lines, moveenabled
up in the fieldset, rename fieldset title "SAML service provider" to "Policy"authentic2/saml/forms.py
there is an akwardraise
added before anothersaml/management/commands/sync-metadata.py
reformat the docstring et remove reference LibertyIdentityProvider in it, the expressednot options['create-disabled']
must not be removed but applied toLibertyProvider.enabled
src/authentic2/saml/migrations/0018_auto_20151208_1542.py
you need to provied an empty backward RunPython migration likedef noop(apps, schema_editor): pass
, see other RunPython migrations in authenticsrc/authentic2/saml/migrations/0020_auto_20151221_1108.py
a RunPython is missing
The you will need to rebase, for testing migrations I advise loading a dump of Montpellier production database and see if it works.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Lié à Project management #10245: Gérer les autorisations de se connecter à un service dans authentic ajouté
Mis à jour par Josué Kouka il y a environ 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch ajouté
Everything seems to be alright. Tested with 3m production db
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
the expressed not options['create-disabled'] must not be removed but applied to LibertyProvider.enabled
This request is not treated.
The noop() operation is not plugged in the RunPython action in migration 0018 and 0020.
Mis à jour par Josué Kouka il y a environ 8 ans
Mis à jour par Josué Kouka il y a environ 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch ajouté
fix typo
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Pour le RunPython, s'il te plait, lis la doc.
Mis à jour par Josué Kouka il y a environ 8 ans
Mis à jour par Josué Kouka il y a environ 8 ans
Benjamin Dauvergne a écrit :
Pour le RunPython, s'il te plait, lis la doc.
Ah oui, c'est
RunPython(FowardFunc, RewardFun)
Mis à jour par Josué Kouka il y a environ 8 ans
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Il faudrait modifier aussi la commande hobo/agent/authentic2/management/commands/hobo_deploy.py
, il y a aussi des mentions de LibertyServiceProvider dans le dépôt authentic2-pratic.git. J'ai fait un grep sur tous les repos je n'en vois pas d'autres (il y a en dans portial-citoyen.git mais ce projet est mort).
Mis à jour par Josué Kouka il y a environ 8 ans
- Lié à Bug #10466: merge des classes A2 LibertyServiceProvider et LibertyProvider ajouté
Mis à jour par Josué Kouka il y a environ 8 ans
- Fichier 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch ajouté
Ajout de la gestion de dépendances avec hobo pour authentic2-multitenant
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Patch proposed changé de Oui à Non
Ce serait vraiment sympa d'intégrer ça à un moment, mais faut rebaser je pense.
Mis à jour par Frédéric Péters il y a 11 mois
- Lié à Development #76893: logguer sur un service SAML incomplet (et/ou revoir les modèles pour que ça n'arrive pas ?) ajouté