Déduplication des fichiers uploadés : référencer l'existant si même taille + même hash #35

New Issue

2026-05-13T22:09:52Z

cedricAbonnel commented

2026-05-13 22:09:52 +00:00

Problème

Chaque upload crée une copie physique du fichier dans data/{uuid}/files/, même si un fichier identique (même contenu) existe déjà dans un autre article. Résultat : espace disque gaspillé, fichiers dupliqués.

Comportement attendu

Lors d'un upload, si un fichier de même hash SHA-256 et même taille existe déjà dans data/*/files/, ne pas stocker de copie supplémentaire : référencer le fichier existant et supprimer le temporaire uploadé.

Contexte technique

Images — naming déjà hash-based

addFile() dans src/ArticleManager.php génère déjà {hash16}-{size}.{ext} pour les images. La détection d'un doublon se réduit à vérifier si ce nom existe dans un autre article :

$existing = glob($this->dataDir . '/*/files/' . $name);
if (!empty($existing)) {
    link($existing[0], $dest) || copy($existing[0], $dest);
    return $name;
}

Pas de recalcul de hash nécessaire — le nom est la clé.

Fichiers non-image

Le nom est basé sur le nom d'origine (sanitisé), pas sur le hash. Options :

calculer le hash SHA-256 et scanner les fichiers existants ;
ou limiter la déduplication aux images en phase 1.

Approche suggérée

Phase 1 : images uniquement — glob sur {hash}-{size}.ext, hard link si trouvé, fallback copy() si filesystems différents.
Phase 2 : non-images via hash calculé à l'upload.

Fichier concerné

app/src/ArticleManager.php — méthode addFile() (~ligne 776)

Notes

Hard link préférable au symlink : résistant aux déplacements, transparent à la lecture.
Ne pas modifier le comportement quand aucun doublon n'est trouvé.

Migré depuis varlog#49

## Problème Chaque upload crée une copie physique du fichier dans `data/{uuid}/files/`, même si un fichier identique (même contenu) existe déjà dans un autre article. Résultat : espace disque gaspillé, fichiers dupliqués. ## Comportement attendu Lors d'un upload, si un fichier de **même hash SHA-256 et même taille** existe déjà dans `data/*/files/`, ne pas stocker de copie supplémentaire : référencer le fichier existant et supprimer le temporaire uploadé. ## Contexte technique ### Images — naming déjà hash-based `addFile()` dans `src/ArticleManager.php` génère déjà `{hash16}-{size}.{ext}` pour les images. La détection d'un doublon se réduit à vérifier si ce nom existe dans un autre article : ```php $existing = glob($this->dataDir . '/*/files/' . $name); if (!empty($existing)) { link($existing[0], $dest) || copy($existing[0], $dest); return $name; } ``` Pas de recalcul de hash nécessaire — le nom est la clé. ### Fichiers non-image Le nom est basé sur le nom d'origine (sanitisé), pas sur le hash. Options : - calculer le hash SHA-256 et scanner les fichiers existants ; - ou limiter la déduplication aux images en phase 1. ## Approche suggérée 1. **Phase 1** : images uniquement — glob sur `{hash}-{size}.ext`, hard link si trouvé, fallback `copy()` si filesystems différents. 2. **Phase 2** : non-images via hash calculé à l'upload. ## Fichier concerné `app/src/ArticleManager.php` — méthode `addFile()` (~ligne 776) ## Notes - Hard link préférable au symlink : résistant aux déplacements, transparent à la lecture. - Ne pas modifier le comportement quand aucun doublon n'est trouvé. --- *Migré depuis [varlog#49](https://git.abonnel.fr/cedricAbonnel/varlog/issues/49)*

cedricAbonnel referenced this issue from cedricAbonnel/varlog

2026-05-13 22:09:52 +00:00

Déduplication des fichiers uploadés : référencer l'existant si même taille + même hash #49

cedricAbonnel closed this issue

2026-05-16 08:57:20 +00:00

cedricAbonnel referenced this issue from a commit

2026-05-16 12:06:32 +00:00

feat : barre de partage articles + déduplication images uploadées (v1.6.20)

Sign in to join this conversation.