Consider adapting lang_handler to use http crate types
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 25/100
- Tipo de issue
- Refatoração
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- rust
- Domínio
- backend, backend-api-design
Direção de pesquisa
Start by reading the current types in lang_handler and comparing them with the http crate and its http-body companion. Define how the extensions API carries socket data, response logs, and exceptions, and assess how the request rewriter can work with the shared types; done means those adaptations are specified and supported without losing the existing needs.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
The http crate provides very similar types as are presently implemented in lang_handler. These could be reused instead to reduce our scope of maintenance. Had I realized that existed I would have just used it from the start. 😅
We would need to use the extensions API to pass through a few additional needed bits like socket data (local/remote ip/port), response logs, and exceptions though, as those do not have any specific support included.
There's a few benefits I see:
- Using known/shared standard code makes maintenance and contribution easier
- In combination with
http-bodycompanion crate, stream support becomes much easier - Our request rewriter could be lifted to be more generalized over http crate types for use in the broader Rust ecosystem
- Linguagem predominante
- Rust
- Estrelas
- 428
- Forks
- 11
- Merge médio
- 29min
- PRs com merge (30d)
- 1
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
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 platformatic/php-node
-
Support for WindowsAberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
platformatic/php-node#58 ·
-
libzip versionAbertabug
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 20/100
platformatic/php-node#43 · 7 comentários ·
-
Create a dev release flowTalvez livre de novo @Qard assumiu há 493 dias e não há nenhum pull request aberto. Aberta
platformatic/php-node#26 · 2 comentários · 1 responsável ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
platformatic/php-node#21 · 2 comentários ·
Todas as issues de platformatic/php-node
Issues semelhantes
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
zcashlabs/thus-spoke-zakura#153 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 79/100
topgrade-rs/topgrade#2395 ·
Mantenedores costumam responder em até 1 dia
-
app bug windows-os
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
matrix-org/matrix-rust-sdk#7217 ·
Mantenedores costumam responder em até 1 dia
-
Improve sublime text syntaxAbertaeditor good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
funnyboy-roks/inq#54 ·