Development #65699
le connecteur SOAP prend les éléments de signature comme des données à envoyer...
0%
Description
Pour calculer le payload qui va être envoyé à Zeep, on fait ceci :
payload = {}
for k in request.GET:
if k == 'raise':
continue
value = request.GET.getlist(k)
if len(value) > 1:
payload[k] = value
else:
payload[k] = value[0]
payload.update(post_data or {})
mais dans request.GET il y a tous les éléments de signature orig, nonce, etc...
Et on se retrouve donc logiquement avec une erreur de schéma, du genre « {http://ws.scrib/}depot() got an unexpected keyword argument 'orig'. Signature: `demandeBacs: {http://ws.scrib/}demandeBacWsDto` »
Je serais pour retirer complétement la prise en charge de la query-string.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 2 ans
- Fichier 0001-soap-do-not-consider-querystring-values-65699.patch 0001-soap-do-not-consider-querystring-values-65699.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 2 ans
- Fichier 0001-soap-do-not-consider-querystring-values-65699.patch 0001-soap-do-not-consider-querystring-values-65699.patch ajouté
Version sans erreur pylint...
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
Ah ben non, pour des référentiels c'est bien pratique, tu veux pas juste ignorer signature, orig, nonce, et toute la clique ?
PS: ou prévoir une convention genre &soap-arg-truc=muche => payload['truc'] = muche@
PS2: et ça revient à supprimer complètement le support du GET en fait....
PS3: voir avec Nico si ça casserait pas un usage déjà en place sur une recette, qu'il puisse migrer ça.
Mis à jour par Frédéric Péters il y a presque 2 ans
PS3: voir avec Nico si ça casserait pas un usage déjà en place sur une recette, qu'il puisse migrer ça.
Je ne capte pas comment ça pourrait déjà être fonctionnel malgré ce bug, on n'a pas partout de la signature automatique sur les appels à des services connus ?
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Fichier 0001-soap-ignore-GET-parameters-not-in-the-schema-65699.patch 0001-soap-ignore-GET-parameters-not-in-the-schema-65699.patch ajouté
- Tracker changé de Bug à Development
counter proposal.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Cool, merci.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2ece5037a939c18b87f3ba622f985209cd79b8df
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed May 25 13:08:27 2022 +0200
soap: ignore GET parameters not in the schema (#65699)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
soap: ignore GET parameters not in the schema (#65699)