NixOS/nix

fetchTree: use last-modified HTTP header where possible

Open

#9.067 aberto em 29 de set. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (1.293 forks)batch import
featurefetchinggood first issue

Métricas do repositório

Stars
 (9.803 stars)
Métricas de merge de PR
 (Mesclagem média 4d 12h) (86 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue a implementação de fetchTree em src/libfetchers/ (provavelmente tarball.cc e file.cc). Procure como lastModified está atualmente definido e onde os cabeçalhos HTTP estão disponíveis. Verifique se a resposta HTTP já passa os cabeçalhos; se não, modifique o fetcher para analisar o cabeçalho last modified e usá lo como fallback quando outro mtime estiver ausente. Considere casos extremos em que o cabeçalho está ausente ou é inválido.
Pilha de tecnologia
cpp
Domain
backendcli
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++HTTP basicsNix source structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50