Développement #105278: wscalls: ne pas mettre en cache ou limiter la durée du cache en cas de code HTTP 4xx/5xx - w.c.s. - Redmine Entr’ouvert

Actions

Copy link

Développement #105278

closed

wscalls: ne pas mettre en cache ou limiter la durée du cache en cas de code HTTP 4xx/5xx

Added by Benjamin Dauvergne 10 months ago. Updated 23 days ago.

Status:

Solution déployée

Priority:

Normal

Assignee:

Benjamin Dauvergne

Target version:

Start date:

30 April 2025

Due date:

% Done:

Estimated time:

Patch proposed:

Planning:

Description

Actuellement si une durée de cache est précisée toutes les réponses quelque soit le code HTTP sont mises en cache, il serait plus utile de ne pas faire de cache ou de limiter la durée du cache à quelque chose uniquement là pour éviter un trop gros impact sur les performances (au pire quelques minutes).

Actions

Copy link

Updated by Benjamin Dauvergne 10 months ago

Assignee set to Benjamin Dauvergne

Actions

Copy link

Updated by Robot Gitea 10 months ago

Status changed from Nouveau to Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312
Titre : wscalls: limit cache to 60 seconds on error status (#105278)
Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/2312/files

Actions

Copy link

Updated by Robot Gitea 10 months ago

Status changed from Solution proposée to En cours

Benjamin Dauvergne (bdauvergne) a commencé à travailler sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312
Titre : WIP: wscalls: limit cache to 60 seconds on error status (#105278)
Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/2312/files

Actions

Copy link

Updated by Robot Gitea 10 months ago

Status changed from En cours to Solution proposée

Actions

Copy link

Updated by Thomas Noël 10 months ago

Status changed from Solution proposée to En cours

Thomas NOËL (tnoel) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

Updated by Benjamin Dauvergne 10 months ago

Status changed from En cours to Solution proposée

Benjamin Dauvergne (bdauvergne) a demandé une relecture de Thomas NOËL (tnoel) sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

Updated by Thomas Noël 10 months ago

Status changed from Solution proposée to En cours

Thomas NOËL (tnoel) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

Updated by Benjamin Dauvergne 10 months ago

Status changed from En cours to Solution proposée

Benjamin Dauvergne (bdauvergne) a demandé une relecture de Thomas NOËL (tnoel) sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

#10

Updated by Thomas Noël 10 months ago

Status changed from Solution proposée to En cours

Thomas NOËL (tnoel) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

#11

Updated by Benjamin Dauvergne 10 months ago

Status changed from En cours to Solution proposée

🤖 Relecture de Thomas NOËL (tnoel) demandée sur la pull request :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

#12

Updated by Thomas Noël 10 months ago

Status changed from Solution proposée to En cours

🤖 Modifications demandées sur la pull request :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

#13

Updated by Benjamin Dauvergne 10 months ago

Status changed from En cours to Solution proposée

🤖 Relecture de Thomas NOËL (tnoel) demandée sur la pull request :

URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2312

Actions

Copy link

#14

Updated by Thomas Noël 9 months ago

Status changed from Solution proposée to En cours

Benjamin Dauvergne a écrit :

🤖 Relecture de Thomas NOËL (tnoel) demandée sur la pull request https://git.entrouvert.org/entrouvert/wcs/pulls/2312

J'ai laissé là bas un commentaire qui indique un soucis avec la couverture des tests : bien tester moment où on va faire du cache parce que la réponse n'est pas du JSON, ou que ça n'est pas un dictionnaire. Je ne comprends pas pourquoi la demande redmine n'est pas repassée en "en cours", j'aurai dû vérifier, sorry

Actions

Copy link

#15