Projet

Général

Profil

Development #30368

nettoyage des form_tokens

Ajouté par Christophe Siraut il y a environ 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Version cible:
-
Début:
04 février 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

des centaines de milliers de fichiers wcs dans le dossier form_tokens; est-ce qu'une purge des vieux fichiers est souhaitable?


Fichiers

Révisions associées

Révision cbd2d357 (diff)
Ajouté par Frédéric Péters il y a presque 5 ans

misc: delete old form_tokens files (#30368)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Tracker changé de Support à Development
  • Projet changé de Admin système à w.c.s.

Ticket à déplacer sur w.c.s., les form_tokens sont nettoyés en même temps que les sessions (je serai pour les créer dans un répertoire avec l'id de la session, ça simplifiera le nettoyage), si il en reste c'est que ce nettoyage a un soucis (surtout je crois que ça ne tient pas compte du fait que quixote ne permet que Sesssion.MAX_FORM_TOKENS dans une même session, soit max 16, quand on dépasse le form_tokens est perdu).

1. se rappeler pourquoi on les crée sur le disque
2. voir si ça ne pourrait pas plutôt utiliser le cache ou rester simplement en session (et voir si besoin d'augmenter MAX_FORM_TOKENS)
3.1 si rien de tout ça implémenter un truc bien simple comme effacer les form_tokens qui ont plus d'1 semaine
3.2 alternative, ajouter le session.id dans le chemin est supprimé tous les nonces pour les ids qui n'existent plus

#2

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Nouveau à Solution proposée
  • Patch proposed changé de Non à Oui

1. se rappeler pourquoi on les crée sur le disque

#16048 qui manque malheureusement de détails pratiques mais vu la zone concernée, je préfère laisser ainsi.

Du coup, option bête, qui prend l'âge max possible des sessions et vire tout ce qui dépasse.

Aussi, lors du job, form_tokens_dir n'était pas défini, parce qu'il est initialisé uniquement lors du set_app_dir(...), quand une requête arrive; je change ça en propriété, comme ça il suffit d'avoir publisher.app_dir, sans nécessairement passer par l'initialisation.

#4

Mis à jour par Emmanuel Cazenave il y a presque 5 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Je ne sais trop où un logger.debug() afficherait/enregistrerait son info, j'ai fait sans.

commit cbd2d3578849dc5373edfed544a090fb394be238
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jun 25 22:45:37 2019 +0200

    misc: delete old form_tokens files (#30368)
#7

Mis à jour par Frédéric Péters il y a presque 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF