Add blacklist headers to protect against DoS attack
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
Línea de trabajo
Comienza en lib/http-proxy/common.js alrededor de la línea 43, donde los encabezados entrantes se incorporan a la solicitud saliente, y revisa el manejo de encabezados salientes de Node.js enlazado. Reproduce la solicitud GET reportada que contiene un encabezado Trailer y verifica que el proxy ya no permita que el error no gestionado resultante termine Node.js. Se considera completado cuando se evita la ruta de DoS reportada y el comportamiento queda cubierto por una prueba de regresión adecuada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hello, I have tried everything to contact the maintainers, nobody answer to me, there is my last try.
-
There is populate headers into outgoing request from incoming:
https://github.com/http-party/node-http-proxy/blob/master/lib/http-proxy/common.js#L43 -
if then add
Trailerheader with any value into the incoming request, that header will be handled by the internal nodejs http lib. For a GET-request, processing that header will trigger unhandled errorERR_HTTP_TRAILER_INVALID. https://github.com/nodejs/node/blob/38cc53845307fdb81dd50cfb7bcfc8c7b83b947c/lib/_http_outgoing.js#L538 -
An unhandled error will cause nodejs shutdown.
If any other project uses node-http-proxy package and just proxy any "user" request, all of them are vulneranilty to DoS attack
- Lenguaje dominante
- JavaScript
- Estrellas
- 14.1k
- Forks
- 2k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de http-party/node-http-proxy
-
util._extend() is depracatoryAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
http-party/node-http-proxy#1719 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
http-party/node-http-proxy#1721 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 55/100
http-party/node-http-proxy#1715 · 1 comentario · 5 reacciones ·
-
Is this library is dead?Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 15/100
http-party/node-http-proxy#1711 · 2 comentarios ·
-
HTTP3 + QUIC support?Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
http-party/node-http-proxy#1710 · 5 reacciones ·
Todos los issues de http-party/node-http-proxy
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
microsoft/TypeScript#65370 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
TMHSDigital/Free-Game-Dev-Assets#272 ·
Los mantenedores suelen responder en 1 día
-
metrics priority: medium
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
LearningCircuit/local-deep-research#8336 ·
Los mantenedores suelen responder en 1 día
-
ace bug yaml
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
svelte: whitespace-only edit leaves an empty todo, edits not trimmed, filter not restored on reloadAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100