Not easy to get the number of methods llhttp has
I maintainer di solito rispondono entro 6 giorni
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 54/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- c, typescript
Direzione di ricerca
Parti da api.c:222 e segui il modo in cui llhttp_method_name gestisce gli ID dei metodi e come viene definita la tabella dei metodi supportati. Esamina l’API esistente e l’esempio aiohttp collegato nell’issue, quindi determina quale comportamento di enumerazione sia appropriato. Il lavoro è completato quando i chiamanti possono scoprire ogni metodo supportato, compresi quelli aggiunti di recente, senza un conteggio codificato staticamente né interrompere la ricerca.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In aiohttp, we need to create an array of all the method names that llhttp supports for caching purposes.
The problem is that llhttp_method_name abort()s on an out-of-range id (api.c:222), our old code therefore just had a hardcoded constant set to 46 and iterated over them. But, this means when llhttp added the QUERY method, our code didn't pick it up.
Would it be reasonable to add something to llhttp to trivially get this information? Either allowing llhttp_method_name to return NULL rather than abort(), or adding a constant that tells us how many methods are available.
Without something like that, I've had to add this awkward thing to extract the names directly:
https://github.com/aio-libs/aiohttp/pull/13174/changes#diff-061b5abe803b659aa544700e4065e1fbff6e83112315e345ea2d2e31ad1d6ca7R106-R117
- Lingua principale
- TypeScript
- Stelle
- 2k
- Fork
- 236
- Merge medio
- 5g 2h
- PR unite (30g)
- 1
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun 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 nodejs/llhttp
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 6 giorni
-
HTTP request smuggling primitive: bare LF accepted as a request-line terminator in strict modeAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
nodejs/llhttp#877 · 1 commento ·
I maintainer di solito rispondono entro 6 giorni
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
nodejs/llhttp#876 · 1 commento ·
I maintainer di solito rispondono entro 6 giorni
-
Should there be a way to block requests with multiple hosts or none are sent?Forse già presa @Vizonex l’ha presa 201 giorni fa. Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 32/100
nodejs/llhttp#816 · 1 commento ·
I maintainer di solito rispondono entro 6 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
nodejs/llhttp#782 · 2 commenti · 4 reazioni ·
I maintainer di solito rispondono entro 6 giorni
Tutte le issue di nodejs/llhttp
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
grafana/profiles-drilldown#1212 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
grafana/traces-drilldown#980 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
openremote/openremote#3487 · 1 commento ·
-
triage:needs-human
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
garrytan/gbrain#7635 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
coollabsio/jean#867 ·
I maintainer di solito rispondono entro 1 giorno