Not easy to get the number of methods llhttp has
Mantenedores costumam responder em até 6 dias
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 54/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- c, typescript
Direção de pesquisa
Comece em api.c:222 e rastreie como llhttp_method_name lida com os IDs dos métodos e como a tabela de métodos compatíveis é definida. Revise a API existente e o exemplo de aiohttp vinculado na issue e, em seguida, determine qual comportamento de enumeração é apropriado. Considera-se concluído quando os chamadores conseguem descobrir todos os métodos compatíveis, incluindo os métodos adicionados recentemente, sem uma contagem codificada ou abortar a busca.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 2k
- Forks
- 236
- Merge médio
- 5d 2h
- PRs com merge (30d)
- 1
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de nodejs/llhttp
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 6 dias
-
HTTP request smuggling primitive: bare LF accepted as a request-line terminator in strict modeAberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
nodejs/llhttp#877 · 1 comentário ·
Mantenedores costumam responder em até 6 dias
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 52/100
nodejs/llhttp#876 · 1 comentário ·
Mantenedores costumam responder em até 6 dias
-
Should there be a way to block requests with multiple hosts or none are sent?Talvez já em andamento @Vizonex assumiu há 202 dias. Aberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 32/100
nodejs/llhttp#816 · 1 comentário ·
Mantenedores costumam responder em até 6 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 45/100
nodejs/llhttp#782 · 2 comentários · 4 reações ·
Mantenedores costumam responder em até 6 dias
Todas as issues de nodejs/llhttp
Issues semelhantes
-
New icon: `filter-clear`Abertacalcite-ui-icons enhancement impact - p3 - not time sensitive needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
Esri/calcite-design-system#15467 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
diegosouzapw/OmniRoute#18390 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 86/100
guilhermerodz/input-otp#157 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
davidjerleke/embla-carousel#1414 ·
Mantenedores costumam responder em até 4 dias