Status code 200 when error in execution handler
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Inizia tracciando la gestione delle eccezioni in Process.py#L232 e il flusso della risposta attraverso Service.py#L326 e Service.py#L335. Riproduci un errore del handler e verifica che la risposta HTTP riporti lo stato 500 anziché 200, mantenendo il comportamento della richiesta non riuscita.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- Python
- Stelle
- 186
- Fork
- 117
- Merge medio
- 17g 23h
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di geopython/pywps
-
ReadTheDocs OAuth App InstallationForse di nuovo libera @tomkralidis l’ha presa 91 giorni fa e non c’è nessuna pull request aperta. Apertadocumentation
-
Modernizing the packaging of the libraryForse di nuovo libera @Zeitsperre l’ha presa 304 giorni fa e non c’è nessuna pull request aperta. Apertadocumentation enhancement packaging
-
DBLog test are bound to other testForse già presa @C1-BA-B1-F3 l’ha presa 16 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
ComplexOutput data as reference always generates a file named "input.csv", should be "output".Forse già presa @C1-BA-B1-F3 l’ha presa 16 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Tutte le issue di geopython/pywps
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
mihai-vlc/sublime-jsfmt#56 ·
-
comp/cli comp/plugins P2 type/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
NousResearch/hermes-agent#141917 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Go device SDK: Whisper transcriber keeps buffering and transcribing audio appended after Stop()Forse già presa @apoorvdarshan l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
BasedHardware/omi#22770 ·
I maintainer di solito rispondono entro 1 giorno
-
bug needs-acceptance wg/router-models-inference-runtime
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
vllm-project/semantic-router#5295 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
huggingface/huggingface_hub#5210 ·
I maintainer di solito rispondono entro 1 giorno