valyala/fasthttp

How to integrate fasthttp client with opentracing gracefully

Open

#775 aberto em 7 de abr. de 2020

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Go (1.755 forks)batch import
help wanted

Métricas do repositório

Stars
 (21.741 stars)
Métricas de merge de PR
 (Mesclagem média 1d 11h) (34 fundiu PRs em 30d)

Description

net/http expose RoundTrip so it can integrate with opentracing by overwrite the RoundTrip function. while fasthttp client don't implement such interface and do not provide middleware. The only way to integrate with opentracing is overwrite all method of fasthttp client, but it is not graceful.

Guia do colaborador

Direção de pesquisa
A issue solicita uma forma elegante de integrar o OpenTracing com o cliente fasthttp, semelhante ao mecanismo RoundTrip do net/http. Pesquise a integração do OpenTracing com net/http em opentracing contrib/go stdlib (nethttp/client.go) para entender a abordagem. Examine o código fonte do cliente fasthttp (client.go) para identificar onde um middleware ou interceptor poderia ser adicionado sem comprometer o desempenho. Considere projetar um wrapper de cliente ou hooks no nível de método. Discuta o design proposto com os mantenedores nos comentários da issue.
Pilha de tecnologia
go
Domain
backendobservability
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go programmingKnowledge of OpenTracing conceptsFamiliarity with fasthttp client
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45