https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342018-03-21T14:16:13ZRedmine Entr’ouvertFargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1058492018-03-21T14:16:13ZBenjamin Dauvergne
<ul><li><strong>Sujet</strong> changé de <i>nettoyer les fichiers orphelins et les obets oauth tempfile</i> à <i>nettoyer les fichiers orphelins et les objets oauth tempfile</i></li></ul> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1062802018-03-23T16:02:59ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/23286">0001-add-a-fargo-cleanup-command-22682.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/23286/0001-add-a-fargo-cleanup-command-22682.patch">0001-add-a-fargo-cleanup-command-22682.patch</a> ajouté</li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1063942018-03-26T12:19:14ZThomas Noël
<ul></ul>Remarques de forme pendant que je cherche à comprendre le fond
<ul>
<li>le created_date devrait sans doute plutôt utiliser auto_now_add que auto_now</li>
<li>indentation sur base de TAB dans le tox.ini </li>
<li>je suis pas sûr que related_name nécessite une "vraie" migration (pas de modif sql ?), on pourrait éventuellement la coller dans la migration précédente (mais je suis pas sûr de mon coup)</li>
<li>max(settings.FARGO_CODE_LIFETIME, settings.FARGO_ACCESS_TOKEN_LIFETIME) : il faudrait poser un commentaire dans le settings.py qui dit que c'est la plus grande qui gagne ? (intuitivement j'aurai pensé à un "min" ici, s'agissant d'une durée de vie, mais j'ai pas bien pigé les conséquences de ce max, en vérité)</li>
</ul> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1064272018-03-26T13:36:48ZBenjamin Dauvergne
<ul></ul><p>Thomas Noël a écrit :</p>
<blockquote>
Remarques de forme pendant que je cherche à comprendre le fond
<ul>
<li>le created_date devrait sans doute plutôt utiliser auto_now_add que auto_now</li>
</ul>
</blockquote>
<p>Mais bien sûr, merci je corrige les deux autres qui ont le même problème dans un pre-patch.</p>
<blockquote>
<ul>
<li>indentation sur base de TAB dans le tox.ini</li>
</ul>
</blockquote>
<p>Ok.</p>
<blockquote>
<ul>
<li>je suis pas sûr que related_name nécessite une "vraie" migration (pas de modif sql ?), on pourrait éventuellement la coller dans la migration précédente (mais je suis pas sûr de mon coup)</li>
</ul>
</blockquote>
<p>Ok.</p>
<blockquote>
<ul>
<li>max(settings.FARGO_CODE_LIFETIME, settings.FARGO_ACCESS_TOKEN_LIFETIME) : il faudrait poser un commentaire dans le settings.py qui dit que c'est la plus grande qui gagne ? (intuitivement j'aurai pensé à un "min" ici, s'agissant d'une durée de vie, mais j'ai pas bien pigé les conséquences de ce max, en vérité)</li>
</ul>
</blockquote>
<p>L'objet porte deux valeurs code et access_token, qui ont des durées de vie différentes, la durée vie du conteneur est forcément le max des deux, sinon on réduit de fait la durée de vie la plus longue à celle la plus courte.</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1064292018-03-26T13:37:04ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/23321">0002-add-a-fargo-cleanup-command-22682.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/23321/0002-add-a-fargo-cleanup-command-22682.patch">0002-add-a-fargo-cleanup-command-22682.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/23322">0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/23322/0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch">0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch</a> ajouté</li></ul><p>Voili voilou.</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1065412018-03-27T08:00:05ZThomas Noël
<ul></ul><p>"Clean expired models of authentic2." : j'imagine que c'est un reste de copié collé, si tu peux mettre quelque chose de plus clair qui explique ce qui est nettoyé (sessions oauth, fichiers orphelins, ... liste à compléter si d'autres modèles ajoutent leur cleanup plus tard).</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1065952018-03-27T09:20:56ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/23334">0002-add-a-fargo-cleanup-command-22682.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/23334/0002-add-a-fargo-cleanup-command-22682.patch">0002-add-a-fargo-cleanup-command-22682.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/23335">0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/23335/0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch">0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch</a> ajouté</li></ul> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1065962018-03-27T09:21:37ZBenjamin Dauvergne
<ul></ul><p>Je ne pense pas que ce soit la peine de spécifier ce qu'on nettoie, c'est forcément des trucs temporaires.</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1066162018-03-27T10:26:51ZThomas Noël
<ul></ul><p>Ack</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1066222018-03-27T11:20:05ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Résolu (à déployer)</i></li></ul><p>Et ça a été poussé et ça a foiré dans jenkins et il y a eu deux commits tapés derrière pour arranger les choses et voilà.</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1066272018-03-27T11:47:59ZBenjamin Dauvergne
<ul></ul><p>Ça me va.</p> Fargo - Development #22682: nettoyer les fichiers orphelins et les objets oauth tempfilehttps://dev.entrouvert.org/issues/22682?journal_id=1099752018-04-24T08:56:57ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Fermé</i></li></ul>