Support for setting keepAliveTimeout
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- docker, node.js
- Área
- backend
Línea de trabajo
El punto de entrada relevante es la configuración del servidor HTTP/HTTPS que crea httpServer y httpsServer; empieza rastreando cómo el contenedor Echo lee las variables de entorno en tiempo de ejecución. Añade un tiempo de espera de keep-alive configurable para ambos servidores, incluido el comportamiento relacionado de headers-timeout mostrado en el issue, y verifica que el contenedor siga iniciándose y aceptando solicitudes HTTP/S con y sin la configuración.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I was making some tests with the Echo container, including ensuring connection pool logics / configuration.
Currently this is taking the default 5s Node keepAliveTimeout which will then start dropping connections. If we would be able to stablish the keepAliveTimeout we would be able to allow the consumer side to manage the connections.
I've hacked a solution as:
// How long an idle keep-alive connection is held before the server closes it.
// Node's default is 5s, so an upstream keep-alive pool in front of this server can never be observed intact.
// Raising it past the drain time lets the pool settle and leaves the reverse proxy in charge of trimming it.
const keepAliveTimeoutMs = parseInt(process.env.KEEPALIVE_TIMEOUT_MS, 10);
if (Number.isFinite(keepAliveTimeoutMs) && keepAliveTimeoutMs > 0) {
for (const server of [httpServer, httpsServer]) {
server.keepAliveTimeout = keepAliveTimeoutMs;
// headersTimeout bounds how long a started request may take to send its headers;
// keeping it above keepAliveTimeout stops Node from closing a pooled connection just as it is being reused.
server.headersTimeout = keepAliveTimeoutMs + 5000;
}
console.log(`Keep-alive timeout set to ${keepAliveTimeoutMs}ms (headers timeout ${keepAliveTimeoutMs + 5000}ms).`);
}
- Lenguaje dominante
- Shell
- Estrellas
- 792
- Forks
- 152
- Merge medio
- 3 d 18 h
- PR fusionados (30 d)
- 3
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 mendhak/docker-http-https-echo
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
mendhak/docker-http-https-echo#86 · 3 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
mendhak/docker-http-https-echo#80 · 2 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
mendhak/docker-http-https-echo#60 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
mendhak/docker-http-https-echo#59 · 1 comentario ·
Todos los issues de mendhak/docker-http-https-echo
Issues similares
-
Issue-Enhancement Needs-Triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
PowerShell/PowerShell#28061 · 2 reacciones ·
-
Feature Request: Add ability to load custom environment variables in linux-exec-server-installer.sh Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
microsoft/vscode-remote-release#11867 ·
-
AuTest Bug Tests
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
apache/trafficserver#13714 ·
-
Update to NCCL 2.32 Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
conda-forge/nccl-feedstock#166 ·
-
Fix codex-seed-model-cache.sh Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
vllm-project/agentic-api#358 · 1 comentario ·