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

Utiliser le cache HTTP pour les fichiers WSDL (#85832) #446

Merged
bdauvergne merged 3 commits from wip/85832-soap-ne-jamais-echouer-un-charge into main 2024-02-04 21:18:55 +01:00
Conversation 3 Commits 3 Files Changed 8 +144 -112
bdauvergne commented 2024-01-22 09:46:22 +01:00
Owner
Copy Link

Au passage rendre le rafraîchissement plus fréquent que l'expiration du cache.

Au passage rendre le rafraîchissement plus fréquent que l'expiration du cache.
bdauvergne added 2 commits 2024-01-22 09:46:23 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
daa5c9887d utils/soap: use http cache for wsdl and xsd files (#85832) 
We should not use another cache since the requests cache has a better
behaviour.
bdauvergne changed title from Utiliser le cache HTTP pour les fichiers WSDL (#85832) to WIP: Utiliser le cache HTTP pour les fichiers WSDL (#85832) 2024-01-22 09:47:10 +01:00
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from daa5c9887d to 70e40d3dca 2024-01-22 18:10:22 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 70e40d3dca to 8a72457aec 2024-01-22 18:10:33 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 8a72457aec to 9704c3ebc9 2024-01-22 18:19:24 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 9704c3ebc9 to 691bc3bc5f 2024-01-22 18:20:19 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 691bc3bc5f to f705395ed2 2024-01-22 18:24:01 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from f705395ed2 to 1b062caa17 2024-01-22 18:48:38 +01:00 Compare
bdauvergne changed title from WIP: Utiliser le cache HTTP pour les fichiers WSDL (#85832) to Utiliser le cache HTTP pour les fichiers WSDL (#85832) 2024-01-22 19:43:37 +01:00
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 1b062caa17 to 6757ef14f3 2024-01-22 19:44:56 +01:00 Compare
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 6757ef14f3 to bb1601d2b0 2024-01-22 19:46:25 +01:00 Compare
bdauvergne commented 2024-01-22 19:46:55 +01:00
Author
Owner
Copy Link

Pour les WSDL on a une durée de conservation du cache d'une semaine et un refresh au minimum toutes 10 minutes, ça n'est pas sensé bouger souvent mais ça bouge ce sera à jour rapidement. Aussi j'arrête d'utiliser le timeout de zeep qui est à 300 secondes pour utiliser le timeout par défaut de passerelle à 25 secondes qui est plus raisonnable.

Pour les WSDL on a une durée de conservation du cache d'une semaine et un refresh au minimum toutes 10 minutes, ça n'est pas sensé bouger souvent mais ça bouge ce sera à jour rapidement. Aussi j'arrête d'utiliser le timeout de zeep qui est à 300 secondes pour utiliser le timeout par défaut de passerelle à 25 secondes qui est plus raisonnable.
bdauvergne requested review from nroche 2024-01-31 21:44:39 +01:00
nroche commented 2024-02-01 11:41:43 +01:00
Owner
Copy Link

J'ai essayé de retirer les scories du cache ZEEP, notamment dans les tests : tests/conftest.py::clear_cache,
mais alors j'ai des erreurs sur les tests test_soap.py et test_utils_soap.py que je n'arrive pas à corriger.

J'ai essayé de retirer les scories du cache ZEEP, notamment dans les tests : tests/conftest.py::clear_cache, mais alors j'ai des erreurs sur les tests test_soap.py et test_utils_soap.py que je n'arrive pas à corriger.
nroche requested changes 2024-02-01 12:20:43 +01:00
nroche left a comment
Owner
Copy Link

Peut-être aussi retirer InMemoryCache des tests.

Peut-être aussi retirer InMemoryCache des tests.
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from bb1601d2b0 to 8d58ae6fe4 2024-02-01 22:44:10 +01:00 Compare
bdauvergne requested review from nroche 2024-02-01 22:47:53 +01:00
bdauvergne commented 2024-02-01 22:48:01 +01:00
Author
Owner
Copy Link

Peut-être aussi retirer InMemoryCache des tests.

C'est fait.

> Peut-être aussi retirer InMemoryCache des tests. C'est fait.
nroche approved these changes 2024-02-02 11:51:16 +01:00
nroche left a comment
Owner
Copy Link

Juste une petite remarque.
Ack, merci pour le boulot.

Juste une petite remarque. Ack, merci pour le boulot.
passerelle/utils/soap.py Outdated
@ -166,14 +163,21 @@ class SOAPTransport(Transport):
def _load_remote_data(self, url):
try:
if urlparse.urlparse(url).netloc != self.wsdl_host:
response = self.session.get(url, timeout=self.load_timeout, auth=None, cert=None)
nroche commented 2024-02-02 11:38:23 +01:00
Owner
Copy Link

On perd le paramètre timeout.
Je pense que ça n'a pas trop d'intérêt de le conserver, mais c'est voulu ?

On perd le paramètre timeout. Je pense que ça n'a pas trop d'intérêt de le conserver, mais c'est voulu ?
bdauvergne commented 2024-02-02 19:48:45 +01:00
Author
Owner
Copy Link

C'est expliqué dans mon commentaire plus haut.

C'est expliqué dans mon commentaire plus haut.
bdauvergne added 1 commit 2024-02-02 20:49:04 +01:00
bdauvergne force-pushed wip/85832-soap-ne-jamais-echouer-un-charge from 9d83916a0b to 0b4de669fd 2024-02-02 20:49:32 +01:00 Compare
bdauvergne merged commit 0b4de669fd into main 2024-02-04 21:18:55 +01:00
bdauvergne deleted branch wip/85832-soap-ne-jamais-echouer-un-charge 2024-02-04 21:18:55 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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#446
No description provided.
Delete Branch "wip/85832-soap-ne-jamais-echouer-un-charge"

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?