- SSRF: /api/queue aceitava qualquer URL, inclusive localhost, rede
privada e 169.254.169.254. Agora só http/https e, por padrão, nada de
endereço local/privado (checagem na fila e no conector do undici, que
cobre DNS rebinding e redirects). ALLOW_PRIVATE_NETWORK=true libera.
- O header x-session-id virava nome de diretório sem validação; com ".."
a expiração faria rmSync em server/. Agora só UUID é aceito.
- Respostas 3xx eram salvas como se fossem o arquivo; agora segue até 5
redirects e trata 3xx restante como erro.
- decodeURIComponent inválido no Content-Disposition derrubava o download.
- web: useCallback em refreshTasks (corrige o warning exhaustive-deps).
O Map de tasks em DownloadManager nunca removia entradas; em um servidor
de longa duração ele crescia sem limite, mesmo com os arquivos da sessão
já apagados. O SessionManager agora avisa quando destrói uma sessão e o
DownloadManager descarta as tasks dela.
Quatro defeitos verificados executando o servidor de produção:
1. `pnpm start` quebrava na primeira linha com ERR_MODULE_NOT_FOUND
(`server/dist/lib/SessionManager`): os imports relativos não tinham extensão
e o Node ESM não resolve `./lib/SessionManager`. Os imports agora terminam em
`.js` (funciona no tsx em dev e no build do tsc).
2. `GET /api/download/:taskId` devolvia o index.html da SPA (450 bytes) em vez
do arquivo baixado, porque o @fastify/static resolve o caminho a partir do
`root` do plugin (web/dist). Agora o root do diretório da sessão é passado
por requisição via `reply.sendFile(fileName, sessionPath)`.
3. O download podia sair truncado: o status virava "completed" logo depois de
`fileStream.end()`, antes do flush em disco. A gravação passou a usar
`stream/promises.pipeline` (respeita backpressure, espera o flush e propaga
erro de escrita), com um Transform no meio mantendo o cálculo de
progresso/velocidade.
4. A raiz de armazenamento era `resolve(process.cwd(), "server/storage")`; como
`pnpm start` executa dentro de `server/`, os arquivos iam para
`server/server/storage`. Agora a raiz é ancorada no diretório do módulo
(`server/storage`), com override por `STORAGE_ROOT`, e o diretório entrou no
.gitignore para os downloads não serem versionados.
Também usa `basename()` no nome de arquivo vindo do Content-Disposition e da
URL de origem, evitando escrever fora do diretório da sessão.
O arquivo server/dist/index.js estava versionado (apesar de `dist/` estar no
.gitignore) e continha o build de novembro/2025, anterior à reescrita do
backend: ainda tinha o proxy antigo de /stream. O build é regenerado por
`pnpm build` (o nixpacks roda `pnpm build` antes do start), então o artefato
não precisa ficar no repositório.
evandro
Evandro Fonseca Junior