grpc-ecosystem/grpc-gateway

Possible regression when specifying duplicate paths

Aperta

#830 aperta il 12 dic 2018

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (0 assegnatari)Go (2250 fork)batch import
bughelp wanted

Metriche repository

Star
 (16.971 stelle)
Metriche merge PR
 (Merge medio 8g 23h) (147 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina le modifiche recenti nella logica di convalida del percorso del generatore grpc gateway. Controlla la cronologia dei commit nel momento in cui è stata introdotta la regressione (probabilmente tra le versioni). Traccia il percorso del codice che gestisce il rilevamento di percorsi duplicati e verifica se un recente refactoring ha rimosso il controllo degli errori.
Tech stack
go
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GogRPC
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40