Développement #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.
Files
Associated revisions
History
Updated by Valentin Deniaud about 2 years ago
- File 0001-auth_saml-remove-metadata-file-path-field-70491.patch 0001-auth_saml-remove-metadata-file-path-field-70491.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Thomas Noël (congés → 5 décembre) about 2 years ago
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)
Updated by Valentin Deniaud about 2 years ago
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 !
Updated by Thomas Noël (congés → 5 décembre) about 2 years ago
- Status changed from Solution proposée to 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 !
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
auth_saml: remove metadata file path field (#70491)