Bug #86217: SAML - Impossible de créer un fournisseur à partir d'un fichier de metadata - Authentic 2 - Redmine Entr’ouvert

Bug #86217

SAML - Impossible de créer un fournisseur à partir d'un fichier de metadata

Ajouté par Benjamin Renard il y a 3 mois. Mis à jour il y a 3 mois.

Statut:

Fermé

Priorité:

Normal

Assigné à:

Frédéric Péters

Catégorie:

SAML

Version cible:

Début:

26 janvier 2024

Echéance:

% réalisé:

Temps estimé:

Patch proposed:

Non

Planning:

Non

Description

Depuis quelque temps, j'ai remarqué qu'il était devenu impossible de créer un fournisseur de service SAML à partir d'un fichier de metadata (champs parcourir). On obtient un message d'erreur indiquant que les meta données sont invalides. En outre, si on copie/colle le contenu du même fichier dans le champ texte, pas de souci. Rien dans les logs pour vous en dire plus, mais vous devriez facilement reproduire ce problème que j'ai pu constater sur plusieurs installations.

Demandes liées

Révisions associées

Révision c5821030 (diff)
Ajouté par Frédéric Péters il y a 3 mois

saml: get uploaded metadata file content as a string (#86217)

Historique

Mis à jour par Frédéric Péters il y a 3 mois

C'est parce que le fichier se trouve lu en bytes, puis comme c'est un CharField il y a un str() dessus et ça donne une chaine type 'b\'

Je vais pousser une PR avec ça :

--- a/src/authentic2/saml/admin.py
+++ b/src/authentic2/saml/admin.py
@@ -61,7 +61,7 @@ class TextAndFileWidget(forms.widgets.MultiWidget):
         # If there is a file input use it
         file = self.widgets[1].value_from_datadict(data, files, name + '_1')
         if file:
-            file = file.read(file.size)
+            file = file.read(file.size).decode()
         if file:
             value = file
         else:

Je suis étonné qu'il n'y ait pas déjà de ticket pour remplacer la vue sous /admin/ par une gestion dans /manage/.