Status code 200 when error in execution handler
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Empieza rastreando el manejo de excepciones en Process.py#L232 y el flujo de respuesta a través de Service.py#L326 y Service.py#L335. Reproduce un error del handler y verifica que la respuesta HTTP informe del estado 500 en lugar de 200, preservando el comportamiento de la solicitud fallida.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
Hey,
I believe there's an issue with the status code. when the handler itself raises an error the Process.py catches it (lines below) sets the wps_request as failed (attached below) but then continues like no error occurred. As result the status code of the HTTP request is 200 and should be 500
I can think of 2 solutions:
- re-raise an NoApplicableCode at Process.py#L232
- before the
return response(Service.py#L326) check whether the response is failed one and if so raise an NoApplicableCode that will be catched at Service.py#L335
What do you guys think? @cehbrecht @jachym
Environment
- operating system: ubuntu 18
- Python version: 3.6
- PyWPS version: latest from github
- source/distribution: git clone
- web server: Flask
- Lenguaje dominante
- Python
- Estrellas
- 186
- Forks
- 117
- Merge medio
- 17 d 23 h
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 geopython/pywps
-
ReadTheDocs OAuth App InstallationQuizá libre de nuevo @tomkralidis la tomó hace 90 días y no hay ningún pull request abierto. Abiertodocumentation
-
Modernizing the packaging of the libraryQuizá libre de nuevo @Zeitsperre la tomó hace 303 días y no hay ningún pull request abierto. Abiertodocumentation enhancement packaging
-
DBLog test are bound to other testPosiblemente ocupada @C1-BA-B1-F3 la tomó hace 16 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
ComplexOutput data as reference always generates a file named "input.csv", should be "output".Posiblemente ocupada @C1-BA-B1-F3 la tomó hace 16 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de geopython/pywps
Issues similares
-
Pull-through from gcr.io returns 500 when pulling using podman/buildahPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoIssue Triage-Needed
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pulp/pulp_container#2581 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
521xueweihan/HelloGitHub#4040 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
ClanGenOfficial/clangen#6313 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
bug(trainer): CustomTrainer func_args with non literal values fail inside the TrainJobPosiblemente ocupada @avanishkasar la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kubeflow/sdk#879 · 1 comentario ·
Los mantenedores suelen responder en 1 día