https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342023-12-12T08:51:51ZRedmine Entr’ouvertCombo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5119072023-12-12T08:51:51ZStéphane Lagetslaget@entrouvert.com
<ul></ul><p>Avec le même besoin sur les cellules json.</p> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5119262023-12-12T09:16:55ZBenjamin Dauvergne
<ul></ul><p>Discuté<sup><a href="#fn1">1</a></sup> récemment en réunion CTP suite à problème similaire au CD06 (#84168-10). Ce serait bien de lier les autres tickets clients ici s'il y en a (et j'en ferai un ticket Publik parce que la solution n'est pas forcément ce nocache).</p>
<p id="fn1" class="footnote"><sup>1</sup> <a class="external" href="https://dev.entrouvert.org/projects/reunions-internes/wiki/Dev-2023-12-11#discussions-plus-larges-quand-il-y-en-a">https://dev.entrouvert.org/projects/reunions-internes/wiki/Dev-2023-12-11#discussions-plus-larges-quand-il-y-en-a</a></p> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5119302023-12-12T09:18:00ZBenjamin Dauvergne
<ul></ul><p>Stéphane Laget a écrit :</p>
<blockquote>
<p>Avec le même besoin sur les cellules json.</p>
</blockquote>
<p>Dans #84168 j'ai aussi identifié le problème avec les expressions <code>{{ cards|...</code> qui utilise aussi un cache par défaut à 15 secondes, dans ce cas il faudrait aussi un filtre <code>|nocache</code> si on reste sur cette idée.</p> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5119312023-12-12T09:24:33ZBenjamin Dauvergne
<ul></ul>J'ai un peu peur qu'un nocache en URL aussi puissant ne donne des facilité pour provoquer des DOS volontaires ou involontaires sur le portail d'une ville; si on prend le cas d'un catalogue d'activité un peu long à lire, le fait de pouvoir désactiver l'utilisation des caches va permettre de provoquer de nombreux appels entre combo/passerelle/le logiciel métier derrière, sans aucun contrôle.<br />On pourrait :
<ul>
<li>n'accepter le filtre que sur une session connectée</li>
<li>poser un timestamp dans le cache ne permettant de ne l'utiliser qu'une fois par seconde/session</li>
</ul>
<p>Il faudrait qu'il agisse sur toutes les instances de requests_wrapper, celle dans combo et celle dans publik-django-templatetags.</p>
<p>PS: explication pour CPF, requests_wrapper c'est le bout de code qui fait les appels HTTPs aux web-services, il en existe une copie dans combo et une dans publik-django-templatetags, c'est à ce niveau qu'est fait le cache, la clé du cache étant l'URL appelée, il se trouve que l'URL appelée par une cellule fiche sur une page récupérant la fiche liée au name_id dans l'URL et par <code>cards|objects:"senior"|filter_by_user:name_id|include_fields|first|get:"user"|get:"mobile"</code> (qui fait la même chose à peu de chose près) n'est pas la même, et donc on ne joue pas sur le même cache.</p> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5122452023-12-12T20:06:52ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> mis à <i>Benjamin Dauvergne</i></li></ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5122462023-12-12T20:07:24ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li></ul><p>Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/combo/pulls/202">https://git.entrouvert.org/entrouvert/combo/pulls/202</a></li>
<li>Titre : WIP: wcs: pass django's request to requests_wrapper (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Pouvoir prévenir combo pour invalider le cache sur une cellule fiches (Fermé)" href="https://dev.entrouvert.org/issues/75916">#75916</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/combo/pulls/202/files">https://git.entrouvert.org/entrouvert/combo/pulls/202/files</a></li>
</ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5181172024-01-10T12:58:03ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5182812024-01-11T08:46:29ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Corentin Sechet (csechet) a approuvé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/combo/pulls/202">https://git.entrouvert.org/entrouvert/combo/pulls/202</a></li>
</ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5192062024-01-15T10:38:31ZOlivier Renard
<ul></ul>Autre cas d'usage:
<ul>
<li>lorsque l'on pose rapidement un commentaire (via un formulaire), le commentaire ne s'affiche pas via la cellule fiche(s) de combo <a class="external" href="https://portail-orenard.test.entrouvert.org/developpements-mutualises-publik/developpement-mutualise/45/">https://portail-orenard.test.entrouvert.org/developpements-mutualises-publik/developpement-mutualise/45/</a> (il faut être authenitifié)</li>
</ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5194242024-01-15T15:19:44ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/combo/pulls/202">https://git.entrouvert.org/entrouvert/combo/pulls/202</a></li>
<li>Titre : pouvoir désactiver le cache de requests_wrapper en ajoutant un paramètre ?nocache à l'URL de la page (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Pouvoir prévenir combo pour invalider le cache sur une cellule fiches (Fermé)" href="https://dev.entrouvert.org/issues/75916">#75916</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/combo/pulls/202/files">https://git.entrouvert.org/entrouvert/combo/pulls/202/files</a></li>
</ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5195702024-01-15T18:14:53ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Combo - Development #75916: Pouvoir prévenir combo pour invalider le cache sur une cellule ficheshttps://dev.entrouvert.org/issues/75916?journal_id=5365322024-03-17T03:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>