grpc-ecosystem/grpc-gateway

Possible regression when specifying duplicate paths

Aberta

#830 aberto em 12 de dez. de 2018

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (0 responsável)Go (2.250 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (16.971 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I haven't got a lot of time to debug this right now, but at work I've recently run into several instances of the generator not erroring when specifying duplicate paths, like so:

get: "/api/v1/users/{user_id}"
...
get: "/api/v1/users/things"

The second one should not be allowed because it is ambiguous with the first one. I'm pretty sure this used to work, so I assume it's a regression. I'm raising just so I don't forget about it. This is happening on latest master.

Guia do colaborador

Direção de pesquisa
Investigue as mudanças recentes na lógica de validação de caminho do gerador grpc gateway. Veja o histórico de commits por volta do momento em que a regressão foi introduzida (provavelmente entre versões). Trace o caminho do código que lida com a detecção de caminhos duplicados e verifique se uma refatoração recente removeu a verificação de erro.
Pilha de tecnologia
go
Domain
backendapi
Tipo Issue
Bug
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GogRPC
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40