fix(server): bloqueia SSRF e path traversal na sessão e segue redirects
- 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).
This commit is contained in:
1 parent
bc9a43d6de
commit
dee0771ac5
7 files changed
+414
-18
No files matched your search
@@ -2,6 +2,7 @@ import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
||||
import { createServer, type Server } from "node:http";
|
||||
import type { AddressInfo } from "node:net";
|
||||
import { existsSync, readdirSync, rmSync } from "node:fs";
|
||||
import { sep } from "node:path";
|
||||
import { Agent } from "undici";
|
||||
import { loadModules } from "./helpers.js";
|
||||
|
||||
@@ -11,6 +12,7 @@ let origin: Server;
|
||||
let originUrl: string;
|
||||
|
||||
beforeEach(async () => {
|
||||
process.env.ALLOW_PRIVATE_NETWORK = "true"; // a origem de teste está em 127.0.0.1
|
||||
vi.useFakeTimers({ toFake: ["setInterval", "Date"] });
|
||||
origin = createServer((_req, res) => {
|
||||
res.setHeader("content-length", "5");
|
||||
@@ -21,6 +23,7 @@ beforeEach(async () => {
|
||||
});
|
||||
|
||||
afterEach(async () => {
|
||||
delete process.env.ALLOW_PRIVATE_NETWORK;
|
||||
vi.useRealTimers();
|
||||
await new Promise((done) => origin.close(done));
|
||||
});
|
||||
@@ -72,3 +75,32 @@ describe("tasks e sessões", () => {
|
||||
rmSync(storageRoot, { recursive: true, force: true });
|
||||
});
|
||||
});
|
||||
|
||||
describe("id de sessão", () => {
|
||||
it("aceita um UUID já conhecido", async () => {
|
||||
const { sessionManager, storageRoot } = await loadModules();
|
||||
const first = sessionManager.getSession();
|
||||
expect(sessionManager.getSession(first.id).id).toBe(first.id);
|
||||
rmSync(storageRoot, { recursive: true, force: true });
|
||||
});
|
||||
|
||||
it.each(["..", ".", "../../etc", "a/b", "not-a-uuid", ""])(
|
||||
"ignora o id inválido %j e cria outra sessão",
|
||||
async (bad) => {
|
||||
const { sessionManager, storageRoot } = await loadModules();
|
||||
const session = sessionManager.getSession(bad);
|
||||
expect(session.id).toMatch(/^[0-9a-f-]{36}$/);
|
||||
expect(sessionManager.getSessionPath(session.id)).toBe(`${storageRoot}${sep}${session.id}`);
|
||||
rmSync(storageRoot, { recursive: true, force: true });
|
||||
},
|
||||
);
|
||||
|
||||
it("não deixa um id '..' apagar a pasta pai quando expira", async () => {
|
||||
const { sessionManager, storageRoot } = await loadModules();
|
||||
sessionManager.getSession("..");
|
||||
vi.setSystemTime(Date.now() + HOUR + 1000);
|
||||
vi.advanceTimersByTime(60_000);
|
||||
expect(existsSync(storageRoot)).toBe(true);
|
||||
rmSync(storageRoot, { recursive: true, force: true });
|
||||
});
|
||||
});
|
||||
Reference in new issue
Block a user