Développement #66243
authenticator FC, exécuter la migration même si A2_FC_AUTHORIZE_URL n'existe pas
0%
Description
Sur un environnement de recette, ce setting peut être implicite, or on se base dessus pour exécuter la migration des données.
Files
Associated revisions
authentic2_auth_fc: fix authenticator creation function (#66243)
History
Updated by Valentin Deniaud over 2 years ago
- File 0002-authentic2_auth_fc-fix-authenticator-creation-functi.patch 0002-authentic2_auth_fc-fix-authenticator-creation-functi.patch added
- File 0001-authentic2_auth_fc-move-authenticator-creation-funct.patch 0001-authentic2_auth_fc-move-authenticator-creation-funct.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Voilà, en se basant plutôt sur la présence du setting A2_FC_ENABLE qui lui est bel et bien requis.
Updated by Paul Marillonnet over 2 years ago
Est-ce qu’on a une idée de combien d’instances sont concernées ? Si c’est genre une ou deux, raccordés au service FC de recette en plus, je suis d’avis qu’on pourrait taper la création de l’authentificateur dans un shell, plutôt que d’ajouter une migration identique à la précédente.
Updated by Frédéric Péters over 2 years ago
je suis d’avis qu’on pourrait taper la création de l’authentificateur dans un shell
L'idée est que les mises à jour de la production fonctionnent, que FC ne se trouve désactivé nulle part, qu'il n'y ait pas un vendredi matin à gérer du support parce que des mires FC auraient disparu.
C'est perso plus rassurant d'avoir ça géré ici que d'avoir à inspecter les prods actuelles pour être sûr que le code sans ce ticket ferait le nécessaire.
Updated by Valentin Deniaud over 2 years ago
Paul Marillonnet a écrit :
Est-ce qu’on a une idée de combien d’instances sont concernées ?
Une bonne quinzaine rien que sur le SaaS.
On pourrait juste corriger la migration, mais c'est « risqué » car alors cette migration tournera pour la première fois en prod. C'est plus rassurant de l'obliger à tourner en en ajoutant une identique, un jour on squashera tout ça et il n'y paraîtra plus :)
Updated by Paul Marillonnet over 2 years ago
Frédéric Péters a écrit :
je suis d’avis qu’on pourrait taper la création de l’authentificateur dans un shell
L'idée est que les mises à jour de la production fonctionnent, que FC ne se trouve désactivé nulle part, qu'il n'y ait pas un vendredi matin à gérer du support parce que des mires FC auraient disparu.
C'est perso plus rassurant d'avoir ça géré ici que d'avoir à inspecter les prods actuelles pour être sûr que le code sans ce ticket ferait le nécessaire.
Ok, j’étais dans l’idée que le caractère implicite du setting manquant ici n’était possible que sur les raccordements au FC d’intégration, sans incidence sur les prod avec un raccordement FC de production effectif, mais j’entends le côté rassurant de passer une nouvelle migration et de ne pas avoir à se poser cette question.
Updated by Paul Marillonnet over 2 years ago
- Status changed from Solution proposée to Solution validée
Valentin Deniaud a écrit :
Paul Marillonnet a écrit :
Est-ce qu’on a une idée de combien d’instances sont concernées ?
Une bonne quinzaine rien que sur le SaaS.
On pourrait juste corriger la migration, mais c'est « risqué » car alors cette migration tournera pour la première fois en prod. C'est plus rassurant de l'obliger à tourner en en ajoutant une identique, un jour on squashera tout ça et il n'y paraîtra plus :)
Ok oui j’entends l’argument, fair enough. Pour la correction de create_fc_authenticator
c’est bon aussi pour moi. Ack.
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit a39e0a8d1b07e03eff7f58343cf659bd11c9015b Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Jun 15 10:21:24 2022 +0200 authentic2_auth_fc: fix authenticator creation function (#66243) commit 522f771159877c9c5cd6e68e66929f1658e74246 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Jun 15 10:20:38 2022 +0200 authentic2_auth_fc: move authenticator creation function (#66243)
Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
authentic2_auth_fc: move authenticator creation function (#66243)