Bug #26417
Dpark: passer le contenu des fichiers en brut à zeep
0%
Description
Les contenu de fichiers sont envoyés en base64 et vu que l'élément censé les contenir est un Base64Binary,
zeep les encode encore une fois en base64.
Fichiers
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 5 ans
- Fichier 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Au niveau du test n'y aurait-il pas moyen de tester véritablement le format en sortie ? Là tu ne testes que ce qui est passé à soap_call, on ne voit pas si l'appel SOAP est correct (j'intercepterai plutôt l'appel à request pour vérifier si le payload est ok selon divers critères, déjà simplement si on y trouve les chaînes recherchées.
Mis à jour par Thomas Noël il y a plus de 5 ans
aussi, le commentaire est pas très heureux « #Use raw content because to prevent zeep from encoding the content into base64 a second time » ça serait plutôt « # Use raw content as zeep encode the content in base64 »
Mis à jour par Josué Kouka il y a plus de 5 ans
- Fichier 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch ajouté
Ok vos commentaires sont pris en compte.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Je ne sais pas pourquoi les gens n'aiment pas httmock (à part le nom pas terrible), enfin bon, je vais tester ce patch.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Josué m'avait demandé mon avis sur la façon de tester ça et je l'avais incité à court-circuiter zeep (dans mon idée, on suppose que zeep marche comme il faut, et on teste uniquement ce qui se passe en amont).
But have it your way.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Assigné à changé de Josué Kouka à Benjamin Dauvergne
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch 0001-dpark-pass-file-s-raw-content-to-zeep-26417.patch ajouté
Ouaips mais bon là c'est pour vérifier ce qui sort donc bon. Partie des tests réécrite à ma sauce, tout roule.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution validée à Fermé
dpark: pass file's raw content to zeep (#26417)