Correction bug FileCache en cas de nombreux fichiers #3184
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Clarified the usage of the #trigger_name# tag and its implications for object, equipment, or command name changes.
Update #trigger_name# tag explanation in scenario.md
Contexte : Dans cache.class.php la fonction persist() de la classe FileCache fait un tar cfz ' . __DIR__ . '/../../cache.tar.gz * 2>&1 > /dev/null; Pourquoi c’est problématique ? Le * est interprété par le shell, pas par tar. Le shell remplace * par tous les fichiers non cachés du répertoire, un par un. Cela peut produire une liste énorme d’arguments, parfois trop grande pour la limite du shell. Pour ma part, avec 22000 fichiers dans mon répertoire cache, mon cache n’était jamais persisté et la page santé de mon jeedom me disait que le cache était en erreur. Comment corriger ? Il suffit dans la commande de remplacer le * par un . : tar cfz ' . __DIR__ . '/../../cache.tar.gz . 2>&1 > /dev/null; Cela permet d’archiver le répertoire courant lui‑même, avec tous ses fichiers et sous‑répertoires, sans dépendre du shell pour l’expansion. → comportement identique → meilleures performances → plus fiable (pas de limite sur le nombre de fichiers, reproductibilité, pas de soucis avec certains noms de fichiers)
scanab
changed the title
Sekiro-kost
requested review from
Hotfirenet,
Salvialf,
tmartinez69009 and
zoic21
zoic21
approved these changes
Jan 8, 2026
kwizer15
approved these changes
Jan 8, 2026
Salvialf
added a commit
that referenced
this pull request
Jan 12, 2026
As same as #3184 without irrelevant documentation changes
Contributor
|
Merci @scanab, Modification directement mise en place en 4.5.3 sans les autres changements qui se sont ajoutés automatiquement lorsque du rebase. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Contexte :
Dans cache.class.php la fonction persist() de la classe FileCache fait un tar cfz ' . DIR . '/../../cache.tar.gz * 2>&1 > /dev/null;
Pourquoi c’est problématique ?
Le * est interprété par le shell, pas par tar. Le shell remplace * par tous les fichiers non cachés du répertoire, un par un. Cela peut produire une liste énorme d’arguments, parfois trop grande pour la limite du shell. Pour ma part, avec 22000 fichiers dans mon répertoire cache, mon cache n’était jamais persisté et la page santé de mon jeedom me disait que le cache était en erreur.
Comment corriger ?
Il suffit dans la commande de remplacer le * par un . : tar cfz ' . DIR . '/../../cache.tar.gz . 2>&1 > /dev/null; Cela permet d’archiver le répertoire courant lui‑même, avec tous ses fichiers et sous‑répertoires, sans dépendre du shell pour l’expansion. → comportement identique
→ meilleures performances
→ plus fiable (pas de limite sur le nombre de fichiers, reproductibilité, pas de soucis avec certains noms de fichiers)
Suggested changelog entry
Related issues/external references
Fixes #
Types of changes
PR checklist