entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 14 Releases Activity

toulouse-maelis: do not crash on tomcat unavailability (#74621) #256

Merged
nroche merged 1 commits from wip/74621-parsifal-receiving-html-content into main 2023-07-07 14:09:33 +02:00
Conversation 0 Commits 1 Files Changed 3 +64 -1
nroche commented 2023-05-22 13:23:25 +02:00
Owner
Copy Link

Je ne reproduis pas l'erreur d'origine (qui concerne la trame reçue), j'imagine parce que ZEEP a évolué depuis.
Mais j'ai une autre erreur (sur la reception du WSDL), j'imagine parce que la gestion des erreurs SOAP dans passerelle a également évoluée.

Ce patch permet d'inscrire dans les tests le contenu HTML que l'on reçoit lorsque les mise à jour de maélis se passent mal.

Je ne reproduis pas l'erreur d'origine (qui concerne la trame reçue), j'imagine parce que ZEEP a évolué depuis. Mais j'ai une autre erreur (sur la reception du WSDL), j'imagine parce que la gestion des erreurs SOAP dans passerelle a également évoluée. Ce patch permet d'inscrire dans les tests le contenu HTML que l'on reçoit lorsque les mise à jour de maélis se passent mal.
nroche force-pushed wip/74621-parsifal-receiving-html-content from 7546673b28 to babb6deb1a 2023-05-22 14:46:35 +02:00 Compare
pmarillonnet requested changes 2023-05-30 14:18:34 +02:00
pmarillonnet left a comment
Owner
Copy Link

Voir mon commentaire ci-dessous.

Voir mon commentaire ci-dessous.
tests/data/toulouse_maelis/R_tomcat_error.html Outdated
@ -0,0 +1,15 @@
<?xml version="1.0" encoding="utf8"?>
pmarillonnet commented 2023-05-30 14:18:24 +02:00
Owner
Copy Link

Il me semble qu’il y a eu erreur dans le copier-coller ici, un mélange du payload SOAP de la requête et de la réponse html de Tomcat. On a un <?xml …> suivi d’un bout de réponse serveur http 404.

Il me semble qu’il y a eu erreur dans le copier-coller ici, un mélange du payload SOAP de la requête et de la réponse html de Tomcat. On a un `<?xml …>` suivi d’un bout de réponse serveur http 404.
nroche commented 2023-06-19 10:37:08 +02:00
Author
Owner
Copy Link

Oui, le bon header étant : <!doctype html>
Il est encore visible dans les logs sur le ticket lié.
j'ai dénaturé la réponse en la passant de mon linter, et du coup j'ai mal traité l'erreur.

J'ai corrigé, merci.
A noter que la sentry donne un cas où la 404 est envoyé après avoir reçu le WSDL (elle plante sur call).
Ici j'en ai profité pour ajouter aussi le cas où l'on a la 404 dès la réception du WSDL.

Oui, le bon header étant : <!doctype html> Il est encore visible dans les logs sur le ticket lié. j'ai dénaturé la réponse en la passant de mon linter, et du coup j'ai mal traité l'erreur. J'ai corrigé, merci. A noter que la sentry donne un cas où la 404 est envoyé après avoir reçu le WSDL (elle plante sur __call__). Ici j'en ai profité pour ajouter aussi le cas où l'on a la 404 dès la réception du WSDL.
nroche force-pushed wip/74621-parsifal-receiving-html-content from babb6deb1a to 4155da062c 2023-06-19 10:25:33 +02:00 Compare
nroche requested review from pmarillonnet 2023-06-19 10:37:20 +02:00
fpeters approved these changes 2023-06-20 10:10:41 +02:00
nroche force-pushed wip/74621-parsifal-receiving-html-content from 4155da062c to 569159a95f 2023-07-07 13:58:38 +02:00 Compare
nroche merged commit 569159a95f into main 2023-07-07 14:09:33 +02:00
nroche deleted branch wip/74621-parsifal-receiving-html-content 2023-07-07 14:09:33 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#256
No description provided.
Delete Branch "wip/74621-parsifal-receiving-html-content"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?