Produits Entr'ouvert » Authentic 2

J'arrive après la bataille mais

ça ouvre une URL en .xml qu'un navigateur permet de visualiser avec de la coloration syntaxique/de télécharger en faisant Ctrl + S

Il faut savoir que cet imbécile de firefox se permet de "corriger" le XML, genre s'amuse à n'afficher aucun attribut xmlns:.. ; même le ctrl-s n'est pas parfaitement égal au fichier original (nettoyage d'espaces et des CRLF, c'est pas méchant, mais quand même).

Bref, c'est pas si méchant, mais il faut le savoir : "voir le xml" dans firefox ne permet pas dire qu'il manque tel ou tel truc, c'est peut-être firefox qui affiche n'importe quoi.

De là, je me dis qu'ajouter une dose supplémentaire de parsing via ElementTree, ça peut encore ajouter le bazar : il serait préférable, selon moi, d'envoyer le contenu de authenticator.metadata "tel quel", pour débusquer le bogue caché tout au fond du xml le jour où.

Je trouve quand même dommage de ne pas renvoyer directement le contenu de authenticator.metadata tel quel (et comme ça pas de "ET.indent"). Mais bon.

À noter dans 0002, ET.indent est introduit en python 3.9, on supporte encore une version inférieure ?

Oui on doit encore gérer buster.

(et oui déjà on a fait n'importe quoi avec la config uwsgi)

Thomas Noël a écrit :

Je trouve quand même dommage de ne pas renvoyer directement le contenu de authenticator.metadata tel quel (et comme ça pas de "ET.indent"). Mais bon.

Après test chez moi sur /idp/saml2/metadata, on ne voit rien du tout à part le base64 de la clé publique désormais, je pense qu'il faut mettre text/plain et ne pas indenter, ou alors faire une vrai vue et présenter ça dans un

<pre>..</pre>

Frédéric Péters a écrit :

J'ai eu un doute sur "désormais" mais ce patch ne change rien ici; chez moi dernier Firefox affiche correctement (formaté etc. cf capture) et au ctrl-s c'est bien le contenu non modifié qui est enregistré. (chromium pareil)

Je n'ai jamais dit que le contenu était modifié à la sauvegarde.

Je serais pour passer un peu de temps à comprendre le comportement de ton navigateur (extensions?) avant de changer/complexifier quoique ce soit.

Ok c'est grammalect qui m'a tué. Mais donc le indent est définitivement inutile ici.

Je n'ai jamais dit que le contenu était modifié à la sauvegarde.

Mon commentaire était plus global que juste le petit bout que j'ai repris, je parlais de ctrl-s parce que ça avait été également questionné dans #70492#note-3.

Frédéric Péters a écrit :

Après test chez moi sur /idp/saml2/metadata, on ne voit rien du tout à part le base64 de la clé publique désormais

J'ai eu un doute sur "désormais" mais ce patch ne change rien ici; chez moi dernier Firefox affiche correctement (formaté etc. cf capture) et au ctrl-s c'est bien le contenu non modifié qui est enregistré. (chromium pareil) Je serais pour passer un peu de temps à comprendre le comportement de ton navigateur (extensions?) avant de changer/complexifier quoique ce soit.

J'ai testé avant de parler, https://perso.entrouvert.org/~thomas/netscaler-metadata.xml n'est pas le contenu du fichier sur mon public_html qui n'est pas non plus exactement ce qu'on obtient avec ctrl-s. Mais c'est pas grave, on s'en fout un peu.u

Pour ma part j'ai juste besoin de voir les metadonnées afin de vérifier qu'elles sont bien les bonnes (parce que sinon, comment ?). Et au final l'affichage bourrin actuel me va, c'est juste qu'il manque un formatage pre../pre. Télécharger le fichier, pas besoin.

J'entends mais « Je serais pour passer un peu de temps à comprendre le comportement de ton navigateur (extensions?) ».

Parce que chez moi, fichier enregistré via ctrl-s vs le fichier d'origine,

 $ md5sum /tmp/netscaler-metadata.xml
251bea090bca66bafcfe98edc0ff12ea  /tmp/netscaler-metadata.xml

 $ ssh leucas.entrouvert.org md5sum ~thomas/public_html/netscaler-metadata.xml
251bea090bca66bafcfe98edc0ff12ea  /home/thomas/public_html/netscaler-metadata.xml

Frédéric Péters a écrit :

J'entends mais « Je serais pour passer un peu de temps à comprendre le comportement de ton navigateur (extensions?) ».

Parce que chez moi, fichier enregistré via ctrl-s vs le fichier d'origine,

[...]

Pareil que Thomas finalement :

--- netscaler-metadata2.xml    2022-10-20 17:53:08.000000000 +0200
+++ netscaler-metadata.xml    2022-10-21 12:02:09.249648768 +0200
@@ -2,14 +2,14 @@
 <md:EntityDescriptor xmlns:md="urn:oasis:names:tc:SAML:2.0:metadata" ID="#_e1f37d80e77061185c7f579d7b1509db" entityID="https://aaa.dordogne.fr">
     <ds:Signature xmlns:ds="https://www.w3.org/2000/09/xmldsig#">
         <ds:SignedInfo xmlns:ds="https://www.w3.org/2000/09/xmldsig#">
-            <ds:CanonicalizationMethod Algorithm="https://www.w3.org/2001/10/xml-exc-c14n#" />
-            <ds:SignatureMethod Algorithm="https://www.w3.org/2000/09/xmldsig#rsa-sha1" />
+            <ds:CanonicalizationMethod Algorithm="https://www.w3.org/2001/10/xml-exc-c14n#"/>
+            <ds:SignatureMethod Algorithm="https://www.w3.org/2000/09/xmldsig#rsa-sha1"/>

Firefox (sans aucune extension activée) enlève les blancs avant les />, c'est pas bien grave, ça ne casserait même pas une signature XML.

Benjamin Dauvergne a écrit :

Firefox (sans aucune extension activée) enlève les blancs avant les />, c'est pas bien grave, ça ne casserait même pas une signature XML.

Oui, on s'en fout. Ce sur quoi j'insiste cependant c'est de retirer le parsing par ElementTree, il ne sert à rien et pourrait même, au pire, un jour, nous induire en erreur. J'insiste un peu car ça sera le seul moyen simple d'accès aux metadata vraiment utilisées par lasso.

Automatic expiration