Development #70491
auth_saml, enlever le champ chemin vers les métadonnées
0%
Description
Qui contient un chemin local sur le serveur, c'est nul.
À la place, migrer les données du fichier en base, dans le champs texte déjà présent.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_saml-remove-metadata-file-path-field-70491.patch 0001-auth_saml-remove-metadata-file-path-field-70491.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus d'un an
Est-ce que logger.exception ça va faire crasher la migration ? Il ne faudrait pas (C'est pas grave si un fichier n'a pas réussi à arriver dans la base, on verra bien que le SAML est tout pété et comme on aura toujours le fichier sur place on pourra réparer à la main)
Mis à jour par Valentin Deniaud il y a plus d'un an
Thomas Noël a écrit :
Est-ce que logger.exception ça va faire crasher la migration ?
Non, logger.exception c'est juste un logger.error avec un machin qui fait qu'il embarque la trace, ça ne fait rien planter du tout (avec un peu de chance ça envoie un mail mais pas sûr sûr).
Il ne faudrait pas
On est d'accord.
C'est pas grave si un fichier n'a pas réussi à arriver dans la base, on verra bien que le SAML est tout pété et comme on aura toujours le fichier sur place on pourra réparer à la main
Pas mieux !
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Thomas Noël a écrit :
Est-ce que logger.exception ça va faire crasher la migration ?
Non, logger.exception c'est juste un logger.error avec un machin qui fait qu'il embarque la trace, ça ne fait rien planter du tout (avec un peu de chance ça envoie un mail mais pas sûr sûr).
Alors c'est parfait, merci !
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 4599cbc739dbe645a6af8c4b2b4847517d7c2fde Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 26 12:09:19 2022 +0200 auth_saml: remove metadata file path field (#70491)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
auth_saml: remove metadata file path field (#70491)