Projet

Général

Profil

Development #8826

Refactoring: merge LibertyServiceProvider into LibertyProvider

Ajouté par Benjamin Dauvergne il y a plus de 8 ans. Mis à jour il y a presque 5 ans.

Statut:
En cours
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
2.2.0
Début:
30 octobre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

liberty_service_provider-to-service_provider-8826.patch (16,5 ko) liberty_service_provider-to-service_provider-8826.patch Josué Kouka, 11 décembre 2015 15:27
0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch (14,8 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch Josué Kouka, 16 décembre 2015 17:44
0001-authentic2-saml-forms-remove-self.childs-there-are-n.patch (1,72 ko) 0001-authentic2-saml-forms-remove-self.childs-there-are-n.patch Benjamin Dauvergne, 18 décembre 2015 12:21
0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch (20,6 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-Libe.patch Josué Kouka, 21 décembre 2015 11:56
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (21,3 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 21 décembre 2015 22:42
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (21,7 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 21 décembre 2015 23:26
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (23,2 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 11 mars 2016 18:05
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (23,4 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 16 mars 2016 17:30
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (23,4 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 17 mars 2016 10:46
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (23,3 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 17 mars 2016 10:56
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (23,2 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 17 mars 2016 11:00
0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch (24,3 ko) 0001-Model-Data-migrations-of-LibertyServiceProvider-to-L.patch Josué Kouka, 05 avril 2016 10:13

Demandes liées

Lié à Publik - Project management #10245: Gérer les autorisations de se connecter à un service dans authenticFermé09 mars 2016

Actions
Lié à Hobo - Bug #10466: merge des classes A2 LibertyServiceProvider et LibertyProviderEn cours29 mars 2016

Actions
Lié à Authentic 2 - Development #76893: logguer sur un service SAML incomplet (et/ou revoir les modèles pour que ça n'arrive pas ?)Nouveau22 avril 2023

Actions

Historique

#1

Mis à jour par Josué Kouka il y a plus de 8 ans

  • Statut changé de Nouveau à En cours
#2

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

  • Version cible mis à 2.2.0
#3

Mis à jour par Josué Kouka il y a plus de 8 ans

#4

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 ?

#5

Mis à jour par Josué Kouka il y a plus de 8 ans

Pour te remettre dans le contexte, l'erreur survenait dans src/authentic2/saml/models.py ligne 34. lasso.Provider

#6

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().

#8

Mis à jour par Josué Kouka il y a plus de 8 ans

LibertyFederation data migration added

#9

Mis à jour par Josué Kouka il y a plus de 8 ans

References to LibertyServiceProvider removed from common.py

#10

Mis à jour par Josué Kouka il y a plus de 8 ans

fix FieldError: Relation fields do not support nested lookups

#11

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

  • authentic2/saml/admin.py remove commented lines, move enabled up in the fieldset, rename fieldset title "SAML service provider" to "Policy"
  • authentic2/saml/forms.py there is an akward raise added before another
  • saml/management/commands/sync-metadata.py reformat the docstring et remove reference LibertyIdentityProvider in it, the expressed not options['create-disabled'] must not be removed but applied to LibertyProvider.enabled
  • src/authentic2/saml/migrations/0018_auto_20151208_1542.py you need to provied an empty backward RunPython migration like def noop(apps, schema_editor): pass, see other RunPython migrations in authentic
  • src/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.

#12

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

#13

Mis à jour par Josué Kouka il y a environ 8 ans

Everything seems to be alright. Tested with 3m production db

#14

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.

#17

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Pour le RunPython, s'il te plait, lis la doc.

#19

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)
#21

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).

#22

Mis à jour par Josué Kouka il y a environ 8 ans

  • Lié à Bug #10466: merge des classes A2 LibertyServiceProvider et LibertyProvider ajouté
#23

Mis à jour par Josué Kouka il y a environ 8 ans

Ajout de la gestion de dépendances avec hobo pour authentic2-multitenant

#24

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.

#25

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Assigné à Josué Kouka supprimé
#26

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é

Formats disponibles : Atom PDF