Projet

Général

Profil

Development #66243

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

Ajouté par Valentin Deniaud il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
14 juin 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Sur un environnement de recette, ce setting peut être implicite, or on se base dessus pour exécuter la migration des données.


Fichiers

Révisions associées

Révision 522f7711 (diff)
Ajouté par Valentin Deniaud il y a presque 2 ans

authentic2_auth_fc: move authenticator creation function (#66243)

Révision a39e0a8d (diff)
Ajouté par Valentin Deniaud il y a presque 2 ans

authentic2_auth_fc: fix authenticator creation function (#66243)

Historique

#3

Mis à jour par Valentin Deniaud il y a presque 2 ans

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

#4

Mis à jour par Paul Marillonnet il y a presque 2 ans

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

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

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

Mis à jour par Valentin Deniaud il y a presque 2 ans

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

Mis à jour par Paul Marillonnet il y a presque 2 ans

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

Mis à jour par Paul Marillonnet il y a presque 2 ans

  • Statut changé de Solution proposée à 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

Mis à jour par Valentin Deniaud il y a presque 2 ans

  • Statut changé de Solution validée à 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

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#12

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF