Suggestion for Handling Empty Data and JSON Decoding in `request_json`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia dal punto di ingresso request_json ed esamina come req:read_cached() fornisce i dati prima che venga chiamato json.decode. Confronta il comportamento con dati vuoti e con JSON malformato, quindi verifica che la risposta vuota prevista e la gestione dell’errore 500 siano coperte senza compromettere la gestione esistente delle richieste.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
- Checking for empty data before attempting decoding, which can prevent unnecessary errors.
- More explicit handling of the case when the data is empty.
For example:
local function request_json(req)
local data = req:read_cached()
if data == '' then
return {}
end
local s, json = pcall(json.decode, data)
if not s then
error(sprintf("Can't decode json in request '%s': %s",
data, tostring(json)))
return nil
end
return json
end
This logic introduces optimization by avoiding unnecessary pcall invocation. It also allows handling a 500 error. However, of course, this requires attention from the developers of tarantool/http. I do not guarantee that the above code will not break some logic.
- Lingua principale
- Lua
- Stelle
- 82
- Fork
- 41
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 tarantool/http
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
-
Too slow performance of HTTPAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
-
1sp code health
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
-
bug question
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 50/100
Tutte le issue di tarantool/http
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
[Operation Enhancement]: .editorconfigと.gitattributesでdictionaryとLuaファイルの編集・改行設定を統一するForse già presa @5kdn l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 80/100
5kdn/DCS-Translation-Japanese#163 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug Report]Apertaarea/Cubelet area/CubeNet bug needs-triage needs:compliance
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
TencentCloud/CubeSandbox#1917 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100