Room for micro-optimisation
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- backend, performance
Línea de trabajo
Start in src/index.js around line 751 and trace the 304 response path together with the readable-stream lifecycle. Done means the 304 check happens before creating the stream and the response does not leave a file descriptor open; verify the behavior with the repository's existing checks.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I'm looking for an asset oriented server. Yours seems great, not tested yet but I saw something weird in the code here : https://github.com/zeit/serve-handler/blob/master/src/index.js#L751
You could make the check before creating the readable stream, 304 will be more efficient, and by the way, you don't destroy it before returning, it may keep open a file descriptor.
Hope it helps,
BR, Ioan
- Lenguaje dominante
- JavaScript
- Estrellas
- 617
- Forks
- 120
- 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 vercel/serve-handler
-
FD leak on 304 Not Modified: opened read stream is never destroyed (leads to EMFILE)Posiblemente ocupada @jaideeppyne la tomó hace 47 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
vercel/serve-handler#235 · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
vercel/serve-handler#231 · 2 comentarios · 5 reacciones ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
vercel/serve-handler#165 · 1 comentario ·
-
Allow serving virtual filesAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
vercel/serve-handler#223 ·
-
Rewrite not working with external url with the portQuizá libre de nuevo @symtalha14 la tomó hace 197 días y no hay ningún pull request abierto. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
vercel/serve-handler#210 · 1 comentario · 3 reacciones ·
Todos los issues de vercel/serve-handler
Issues similares
-
Site Suggestion: bullseyeAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 25 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
libnativeapi/nativeapi#21 ·
-
area: core complexity: easy impact: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
ueberdosis/tiptap#8668 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día