Sanitizing input text for "get" queries?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Reproduce el fallo con el endpoint /study/1/samples y el valor de term mostrado en el issue. Rastrea el handler del servidor para esa consulta GET y determina cómo la entrada entre comillas llega a la operación que falla; se considera terminado cuando la solicitud ya no devuelve un 500 para este ID de espécimen y el comportamiento esperado de la consulta está cubierto por una prueba de regresión.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Doesn't look like this has been logged as an issue before, so I'm documenting it here.
In the style of xkcd, I tried plating a specimen with the ID ');console.log('hi');. Fortunately, this doesn't result in hi being printed to the console; unfortunately, it does cause a 500 internal server error:
GET https://127.0.0.1:8181/study/1/samples?term=%27);console.log(%27hi%27); 500 (Internal Server Error)
Looks like it's the ' character (which gets converted to %27) that's problematic.
I don't imagine this is a common use case, but it might be worth addressing?
Edit: also, if this behavior is expected (i.e. the server throws an error if it detects a wacky specimen ID like ');console.log('hi');) then sorry, and feel free to close this.
- Lenguaje dominante
- Python
- Estrellas
- 2
- Forks
- 15
- 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 biocore/LabControl
-
In plating interface, valid sample IDs occasionally "flash" red before turning another colorAbiertobug front-end question
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
biocore/LabControl#594 ·
-
front-end question
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
biocore/LabControl#593 ·
-
Cache list of active samples when the active study is changed in the plating interfaceQuizá libre de nuevo @fedarko la tomó hace 2588 días y no hay ningún pull request abierto. Abiertofront-end
biocore/LabControl#592 · 1 asignado ·
-
code refactor front-end
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
biocore/LabControl#591 ·
-
priority:low
Dificultad 3/5 1-2 días Aptitud para principiantes 20/100
biocore/LabControl#590 ·
Todos los issues de biocore/LabControl
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
vllm-project/vllm#61131 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kubernetes-client/python#2735 ·
Los mantenedores suelen responder en 1 día
-
desktop help wanted test
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Broken links found in docsAbiertodocs pydanty:is-working
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pydantic/pydantic-ai#10113 ·
Los mantenedores suelen responder en 1 día
-
Tutorial unit test leaks sample data into tutorial doctestPosiblemente ocupada @Gambit-Checkmate la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100