labstack / labstack/echo

Group routes does not work without leading slash

Offen Anfängerfreundlich
#3,099 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

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

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Ö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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.