Project

General

Profile

Développement #66243

authenticator FC, exécuter la migration même si A2_FC_AUTHORIZE_URL n'existe pas

Added by Valentin Deniaud over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
14 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 522f7711 (diff)
Added by Valentin Deniaud over 2 years ago

authentic2_auth_fc: move authenticator creation function (#66243)

Revision a39e0a8d (diff)
Added by Valentin Deniaud over 2 years ago

authentic2_auth_fc: fix authenticator creation function (#66243)

History

#3

Updated by Valentin Deniaud over 2 years ago

Voilà, en se basant plutôt sur la présence du setting A2_FC_ENABLE qui lui est bel et bien requis.

#4

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.

#5

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.

#6

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

#7

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.

#8

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.

#9

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

Updated by Transition automatique over 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#12

Updated by Transition automatique over 2 years ago

Automatic expiration

Also available in: Atom PDF