Testing: add a regression test that proves ?token= never leaks into access logs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Lee el middleware de registro de solicitudes y las pruebas existentes del registro de acceso para encontrar dónde se eliminan las cadenas de consulta y se enmascaran los valores de los tokens. Añade una prueba de regresión específica que envíe una solicitud con ?token= y compruebe el registro de acceso capturado. La tarea está terminada cuando el valor del token no aparece y la solicitud registrada no tiene cadena de consulta.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
There is no automated regression test proving that ?token= is removed from request logs and access-log formatting continues to mask token values. Since the query-token fallback is a key compatibility feature, it deserves a direct check.
Reproduction
- Run the server with access logging enabled.
- Send requests using
?token=. - Inspect logs manually.
- There is currently no automated check guarding this behavior.
Expected
A focused test should assert that token values do not appear in access logs and that the middleware strips the query string before logging.
Actual
This remains a manual assumption.
- Lenguaje dominante
- Python
- Estrellas
- 22
- Forks
- 4
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 openapi/mcp-server
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
openapi/mcp-server#29 ·
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 73/100
openapi/mcp-server#25 ·
-
Testing: add a host-agnostic smoke test checklist for OpenClaw, NanoClaw, and NemoClaw-style clientsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
openapi/mcp-server#24 ·
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
openapi/mcp-server#21 ·
-
enhancement good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
openapi/mcp-server#19 ·
Todos los issues de openapi/mcp-server
Issues similares
-
md/asciidoc/csv: an explicit `encoding="utf-8"` keeps a leading BOM, the auto-detect path drops itAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
docling-project/docling#4846 ·
Los mantenedores suelen responder en 1 día
-
area/sessions comp/dashboard comp/desktop comp/gateway comp/tui duplicate P2 sweeper:risk-session-state type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
NousResearch/hermes-agent#138914 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bot-triaged module: activation checkpointing module: inductor oncall: pt2 triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Update PR Actions Detective E2E prerequisites and fix dashboard-format table typoAbiertodocumentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
elastic/oblt-aw#2382 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día