NixOS/nix

fetchTree: use last-modified HTTP header where possible

Open

#9067 aperta il 29 set 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (1293 fork)batch import
featurefetchinggood first issue

Metriche repository

Star
 (9803 star)
Metriche merge PR
 (Merge medio 4g 12h) (86 PR mergiate in 30 g)

Descrizione

Is your feature request related to a problem? Please describe.

Currently fetchTree with type=tarball+http(s) gets lastModified from the mtime of the tarball contents. Some formats such as zip do not include this metadata, so lastModified ends up being incorrectly set to the current time.

type=file+http(s) does not set lastModified at all.

Describe the solution you'd like

Most webservers, when serving static files, will send a last-modified header set to the mtime of the served file. We could set lastModified to this value, or use it as fallback when mtime of contents is absent.

Describe alternatives you've considered

Do not set lastModified when mtime metadata is not present in archives. This should probably be done anyway when neither mtime nor last-modified is present.

Additional context

Priorities

Add :+1: to issues you find important.

Guida contributor

Direzione di ricerca
Indaga l'implementazione di fetchTree in src/libfetchers/ (probabilmente tarball.cc e file.cc). Cerca come lastModified è attualmente impostato e dove sono disponibili gli header HTTP. Controlla se la risposta HTTP passa già gli header; in caso contrario, modifica il fetcher per analizzare l'header last modified e usarlo come fallback quando altri mtime sono assenti. Considera i casi limite in cui l'header è mancante o non valido.
Tech stack
cpp
Dominio
backendcli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++HTTP basicsNix source structure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50