Group routes does not work without leading slash
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 32.7k
- Forks
- 2.8k
- Ø Merge
- 9 Std. 35 Min.
- Gemergte PRs (30 T.)
- 6
Beschreibung
Group route does not work without leading slash
I ran into an unexpected behavior with grouped routes.
g := e.Group("/v1")
g.GET("posts", handler)
Requesting:
GET /v1/posts
returns:
Not Found
Changing it to:
g.GET("/posts", handler)
works as expected.
I would expect Echo to handle the missing leading / when combining the group prefix and route path, or at least provide a warning during route registration.
It would be nice if both forms resulted in /v1/posts:
g.GET("posts", handler)
g.GET("/posts", handler)
Because it is very normal for a developer to not provide the slash since a group already means it's under that group (so slash should come automatically)
Otherwise its too confusing and something to forget easily when using the framework!!
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, das Beispiel für gruppierte Routen im Issue zu reproduzieren, und verfolge die Registrierung und das Matching-Verhalten von Echo für Gruppenrouten. Füge Regressionstests sowohl für g.GET("posts", handler) als auch für g.GET("/posts", handler) unter /v1 hinzu; als erledigt gilt die Aufgabe, wenn Anfragen an /v1/posts in beiden Fällen aufgelöst werden oder das dokumentierte Warnverhalten abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- go
- Bereich
- api, backend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 74/100