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

soap: activer api_error=True sur Parsifal (#73411) #37

Closed
nroche wants to merge 3 commits from wip/73411-maelis-manage-soap-error into main
pull from: wip/73411-maelis-manage-soap-error
merge into: entrouvert:main
Conversation 6 Commits 3 Files Changed 4 +71 -6
nroche commented 2023-01-13 21:00:06 +01:00
Owner
Copy Link

0001: J'ai isolé l'erreur.

La correction est donnée dans https://dev.entrouvert.org/issues/65763
Je ne l'ai d'abord pas compris, ce qui a donné suite à beaucoup de tickets pour valider les paramètres issus des référentiels, dont on aurait pu se passer, mais qui a quand même permis de clarifier certains messages d'erreurs).
En l'activant (0002), on fait crasher l'encodeur json sur du contenu XML qui apparaît dans un champ "detail" de l'exception.

Ce que répare 0003, en prenant en compte le nouveau champ "detail".

0001: J'ai isolé l'erreur. La correction est donnée dans https://dev.entrouvert.org/issues/65763 Je ne l'ai d'abord pas compris, ce qui a donné suite à beaucoup de tickets pour valider les paramètres issus des référentiels, dont on aurait pu se passer, mais qui a quand même permis de clarifier certains messages d'erreurs). En l'activant (0002), on fait crasher l'encodeur json sur du contenu XML qui apparaît dans un champ "detail" de l'exception. Ce que répare 0003, en prenant en compte le nouveau champ "detail".
nroche added 3 commits 2023-01-13 21:00:07 +01:00
gitea-wip/passerelle/pipeline/pr-main This commit looks good Details
932907630b toulouse-maelis: manage XML soap fault detail content (#73411)
nroche changed title from wip/73411-maelis-manage-soap-error to soap: activer api_error=True sur Parsifal (73411) 2023-01-13 21:00:36 +01:00
nroche changed title from soap: activer api_error=True sur Parsifal (73411) to soap: activer api_error=True sur Parsifal (#73411) 2023-01-13 21:48:42 +01:00
bdauvergne commented 2023-01-16 10:28:55 +01:00
Owner
Copy Link

On ne peut pas commenter sur les fichiers ça dit " nroche veut fusionner 0 révision(s)", donc le merge est vide.

On ne peut pas commenter sur les fichiers ça dit " nroche veut fusionner 0 révision(s)", donc le merge est vide.
bdauvergne commented 2023-01-16 11:02:42 +01:00
Owner
Copy Link

Il faudrait ne pas modifier fault.detail, en copiant directement fault.dict en premier.

Il faudrait ne pas modifier fault.detail, en copiant directement fault.__dict__ en premier.
nroche commented 2023-01-17 17:39:00 +01:00
Author
Owner
Copy Link

On ne peut pas commenter sur les fichiers ça dit " nroche veut fusionner 0 révision(s)", donc le merge est vide.

Grosse erreur de ma part : de mémoire, j'ai poussé main sur gitea (mon historique est mélangé je ne retrouve pas).
Puis j'ai voulu réparer en repoussant main sur gitmain (sans mes patchs), mais j'ai dû oublier de forcer.

Résultat, je m'aperçois que mes 3 patchs sont poussés en recette.
Dis-moi si je réverte (ou si je corrige via le patch ajouté, mais je pense qu'il y a peu de chance que j'ai compris ta demande de correction).

> On ne peut pas commenter sur les fichiers ça dit " nroche veut fusionner 0 révision(s)", donc le merge est vide. Grosse erreur de ma part : de mémoire, j'ai poussé main sur gitea (mon historique est mélangé je ne retrouve pas). Puis j'ai voulu réparer en repoussant main sur gitmain (sans mes patchs), mais j'ai dû oublier de forcer. Résultat, je m'aperçois que mes 3 patchs sont poussés en recette. Dis-moi si je réverte (ou si je corrige via le patch ajouté, mais je pense qu'il y a peu de chance que j'ai compris ta demande de correction).
bdauvergne commented 2023-01-18 17:42:30 +01:00
Owner
Copy Link

Revert please.

Revert please.
nroche force-pushed wip/73411-maelis-manage-soap-error from daba6702f2 to 39ee73393a 2023-01-18 18:45:04 +01:00 Compare
nroche commented 2023-01-18 18:49:41 +01:00
Author
Owner
Copy Link

Revert fait et branche repoussée ici avec la correction demandée telle que je l'ai interprétée.
(j'ai laissé les 3 commits parce que je traite 2 problèmes conséquents, mais je pensais squasher ensuite).

Revert fait et branche repoussée ici avec la correction demandée telle que je l'ai interprétée. (j'ai laissé les 3 commits parce que je traite 2 problèmes conséquents, mais je pensais squasher ensuite).
nroche force-pushed wip/73411-maelis-manage-soap-error from 39ee73393a to 1695310c54 2023-01-18 19:02:49 +01:00 Compare
nroche closed this pull request 2023-02-07 18:31:50 +01:00
nroche deleted branch wip/73411-maelis-manage-soap-error 2023-02-07 18:33:03 +01:00
nroche commented 2023-02-07 18:42:51 +01:00
Author
Owner
Copy Link

Nouveau patch dans la PR #96.

Nouveau patch dans la PR #96.
All checks were successful
gitea-wip/passerelle/pipeline/pr-main This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#37
No description provided.
Delete Branch "wip/73411-maelis-manage-soap-error"

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?