fail to deal with request header Expect: 100-Continue and respone
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- lua, nginx
- Ambito
- networking
Direzione di ricerca
Inizia in lib/resty/http.lua alle righe indicate intorno a _handle_continue e ai controlli dell'header Expect. Confronta la gestione dei valori Expect senza distinzione tra maiuscole e minuscole, degli header opzionali nelle risposte 100 e delle risposte 100 non richieste con le sezioni RFC citate. Il lavoro è completato quando tutti e tre i casi segnalati vengono gestiti correttamente e i test o le riproduzioni esistenti hanno esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
hi @pintsized ,
Bug 1
the following point is case sensitive,so if the value of request header Expect has some upper character,we will make mistake without handle_continue;
Bug 2
_handle_continue only try to read an empty line,but there is some oter optional headers like Date or Content-length. see rfc2616-10.1 Informational 1xx.
Bug 3
http.lua#L557 only handle continue response when Expect: 100-continue request header is there.but as the RFC 2616 say:
- An origin server SHOULD NOT send a 100 (Continue) response if
the request message does not include an Expect request-header
field with the "100-continue" expectation, and MUST NOT send a
100 (Continue) response if such a request comes from an HTTP/1.0
(or earlier) client.
see RFC2616#sec8.2.3.
It's SHOULD NOT requirement,it would be nice if we support it.
I have made a fix and tested it ok.
thanks.
- Lingua principale
- Lua
- Stelle
- 2.1k
- Fork
- 629
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna 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 ledgetech/lua-resty-http
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
ledgetech/lua-resty-http#337 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
ledgetech/lua-resty-http#334 · 1 commento · 1 reazione ·
-
Follow Redirect FeatureAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
ledgetech/lua-resty-http#330 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 15/100
ledgetech/lua-resty-http#325 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
ledgetech/lua-resty-http#324 ·
Tutte le issue di ledgetech/lua-resty-http
Issue simili
-
Add event count to export UIApertaFeature Request
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Questie/QuestieTrace#92 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
WitchAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
LuaLS/lua-language-server#3471 ·
-
RadlNavi Routing V2.3.5 Rechts/Links Kombinationen halb unterdrücktForse già presa @thomas-munichways l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
bug good first issue ready ticket
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Domktorymysli/vclu#20 ·