Skip to content

feat(Bimex): security-indexer-l-mite-de-64kb-en-cuerpos-de-re - #285

Closed
Caneryy wants to merge 1 commit into
David1984TK:mainfrom
Caneryy:feat/issue-236
Closed

feat(Bimex): security-indexer-l-mite-de-64kb-en-cuerpos-de-re#285
Caneryy wants to merge 1 commit into
David1984TK:mainfrom
Caneryy:feat/issue-236

Conversation

@Caneryy

@Caneryy Caneryy commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Closes #236

Summary

  • security(indexer): límite de 64KB en cuerpos de request + timeouts anti-slowloris

Test plan

  • Relevant tests pass locally
  • Manual verification on affected UI or API paths

@Caneryy
Caneryy requested a review from David1984TK as a code owner July 29, 2026 00:51
@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown

@Caneryy is attempting to deploy a commit to the david1984tk's projects Team on Vercel.

A member of the Team first needs to authorize it.

@David1984TK

Copy link
Copy Markdown
Owner

Cerrando: el límite de 64KB ya está en main desde #282. Además este PR reintroduce un bug que encontré y corregí ahí — vuelve a llamar req.destroy() inmediatamente después de json(...) en vez de esperar res.once('finish', ...), lo que corta el socket antes de que el cliente reciba la respuesta 413 (queda como socket hang up en vez del error real, lo verifiqué con tests). Gracias por el aporte de todas formas — los tests de headersTimeout/requestTimeout que agregás son una buena idea, si querés mandalos en un PR aparte contra el código actual.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

security(indexer): límite de 64KB en cuerpos de request + timeouts anti-slowloris

2 participants