borgbackup/borg

unusual blake2b usage, adopt blake3?

Open

#8.867 geöffnet am 21. Mai 2025

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (10.086 Stars) (732 Forks)batch import
breakingcmd: transferhelp wantedsecurity

Beschreibung

This is from master branch (same in 1.4-maint):

def blake2b_256(key, data):
    return hashlib.blake2b(key+data, digest_size=32).digest()

It is one of the chunkid hashes borg uses.

The usage there is unusual, the python docs rather suggest:

def blake2b_256(key, data):
    return hashlib.blake2b(data, key=key, digest_size=32).digest()

Contributor Guide

Tech Stack
python
Domain
backend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Python hashlibcryptographic hashing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
The issue discusses an unusual usage of hashlib.blake2b in the borg codebase, specifically in the blake2b 256 function that generates chunk IDs. There are two possible paths: correcting the call to use the key parameter properly, or adopting blake3 for potentially better performance. The issue has an assignee, so any contribution should first coordinate with them. The relevant code is likely in borg's hash or crypto module.