Projet

Général

Profil

Development #50260

astregs: retirer automatiquement les caractères avant le contenu XML et logger un warning dans ce cas

Ajouté par Benjamin Dauvergne il y a plus de 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
18 janvier 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On a régulièrement des soucis et ce ne sera visiblement pas corrigé suffisamment vite; ce problème obscurci les vrais soucie en cachant les erreurs remontées par AstreGS.


Fichiers

0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch (4,57 ko) 0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch Benjamin Dauvergne, 29 janvier 2021 16:28
0001-tests-code-style-50260.patch (1,8 ko) 0001-tests-code-style-50260.patch Benjamin Dauvergne, 29 janvier 2021 16:28
0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch (4,57 ko) 0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch Benjamin Dauvergne, 29 janvier 2021 19:45
0003-astregs-use-remove_first_bytes_for_xml-50260.patch (1,06 ko) 0003-astregs-use-remove_first_bytes_for_xml-50260.patch Benjamin Dauvergne, 29 janvier 2021 19:45
0001-tests-code-style-50260.patch (1,8 ko) 0001-tests-code-style-50260.patch Benjamin Dauvergne, 29 janvier 2021 19:45
0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch (4,56 ko) 0002-misc-allow-fixing-prefix-characters-in-SOAP-response.patch Benjamin Dauvergne, 01 février 2021 11:21
0003-astregs-use-remove_first_bytes_for_xml-50260.patch (1,06 ko) 0003-astregs-use-remove_first_bytes_for_xml-50260.patch Benjamin Dauvergne, 01 février 2021 11:21
0001-tests-code-style-50260.patch (1,8 ko) 0001-tests-code-style-50260.patch Benjamin Dauvergne, 01 février 2021 11:21
0002-utils-soap-ignore-content-before-and-after-SOAP-XML-.patch (4,67 ko) 0002-utils-soap-ignore-content-before-and-after-SOAP-XML-.patch Benjamin Dauvergne, 01 février 2021 12:04
0003-astregs-use-remove_first_bytes_for_xml-50260.patch (1,06 ko) 0003-astregs-use-remove_first_bytes_for_xml-50260.patch Benjamin Dauvergne, 01 février 2021 12:04
0001-tests-code-style-50260.patch (1,8 ko) 0001-tests-code-style-50260.patch Benjamin Dauvergne, 01 février 2021 12:04

Révisions associées

Révision e70576bc (diff)
Ajouté par Benjamin Dauvergne il y a environ 3 ans

tests: code style (#50260)

Révision 090c2f40 (diff)
Ajouté par Benjamin Dauvergne il y a environ 3 ans

utils/soap: ignore content before and after SOAP XML content (#50260)

Révision 72ee589c (diff)
Ajouté par Benjamin Dauvergne il y a environ 3 ans

astregs: use remove_first_bytes_for_xml (#50260)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Assigné à mis à Benjamin Dauvergne
#4

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

C'est pour régler les soucis comme là, #50182#note-4

#5

Mis à jour par Nicolas Roche il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

ok.

#7

Mis à jour par Nicolas Roche il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

(pour moi c'est toujours bon)

#9

Mis à jour par Serghei Mihai il y a environ 3 ans

En pratique quand la réponse d'Astre est foireuse, il y a aussi du texte après le bout XML:

Server returned response (500) with invalid XML: Invalid XML content received (Start tag expected, '<' not found, line 1, column 1).
Content: b'1a5\r\n<?xml version=\'1.0\' encoding=\'utf-8\'?><soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><soapenv:Fault><faultcode>soapenv:Server</faultcode><faultstring>Erreur technique dans le service \'DossierIndicateur\'&#xd;\nErreur WebService : L&amp;#39;indicateur est deja li\xc3\xa9 au dossier pour l&amp;#39;ann\xc3\xa9e saisie.</faultstring><detail /></soapenv:Fault></soapenv:Body></soapenv:Envelope>\r\n0\r\n\r\n'

#10

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

Serghei Mihai a écrit :

En pratique quand la réponse d'Astre est foireuse, il y a aussi du texte après le bout XML:
[...]

Yep tu as raison, j'ouvre un autre ticket.

PS: je pensais que j'avais poussé mais non.

#12

Mis à jour par Nicolas Roche il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

(vu le bloublou)

#13

Mis à jour par Nicolas Roche il y a environ 3 ans

(je ne pense pas que ce soit pénalisant, mais j'ai rien vu concernant la partie "et logger un warning dans ce cas")

#14

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

Nicolas Roche a écrit :

(je ne pense pas que ce soit pénalisant, mais j'ai rien vu concernant la partie "et logger un warning dans ce cas")

Finalement je me suis dit que vu qu'on sait que le truc fait ça et le fera peut-être toujours le warning n'est pas super important. Au début je me suis dit que j'allais faire ça pour tous les connecteurs et là le warning aurait eu tout son sens.

#15

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 72ee589cb00b0d4fd36c1d1910b1a406f49f4e93
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Jan 29 19:44:49 2021 +0100

    astregs: use remove_first_bytes_for_xml (#50260)

commit 090c2f4062e5241bab8e8e89d8793214469f593c
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Jan 29 16:02:09 2021 +0100

    utils/soap: ignore content before and after SOAP XML content (#50260)

commit e70576bc6ce1d6e8bcebaf0fb377657d9826df3a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Jan 29 16:01:47 2021 +0100

    tests: code style (#50260)

#16

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF