Group routes does not work without leading slash
まだ誰も着手していません。
評価
調査の方向性
まず issue のグループ化されたルートの例を再現し、Echo のグループルートの登録とマッチングの動作を追跡します。/v1 配下の g.GET("posts", handler) と g.GET("/posts", handler) の両方について回帰テストを追加します。完了条件は、どちらの場合も /v1/posts へのリクエストが解決されること、または文書化された警告の動作がカバーされていることです。
索引モデルが issue の本文から書いたものです。
説明
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!!
- 主要言語
- Go
- スター
- 32.7k
- フォーク
- 2.8k
- 平均マージ
- 9時間 39分
- マージ済み PR(30日)
- 6
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
labstack/echo のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 74/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 60/100
github/gh-aw-mcpg#13748 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
googleapis/librarian#7670 · コメント 2 件 ·