Project

General

Profile

Development #70491

auth_saml, enlever le champ chemin vers les métadonnées

Added by Valentin Deniaud 3 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
19 October 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 4599cbc7 (diff)
Added by Valentin Deniaud 3 months ago

auth_saml: remove metadata file path field (#70491)

History

#1

Updated by Valentin Deniaud 3 months ago

#2

Updated by Thomas Noël 3 months 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)

#3

Updated by Valentin Deniaud 3 months 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 !

#4

Updated by Thomas Noël 3 months 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 !

#5

Updated by Valentin Deniaud 3 months 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)
#6

Updated by Transition automatique about 1 month ago

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

Also available in: Atom PDF