Skip to content

fix: join group routes without leading slash - #3104

Closed
Rayan-and-beyond wants to merge 1 commit into
labstack:masterfrom
Rayan-and-beyond:fix/group-route-leading-slash
Closed

Rayan-and-beyond wants to merge 1 commit into
labstack:masterfrom
Rayan-and-beyond:fix/group-route-leading-slash

Conversation

@Rayan-and-beyond

Copy link
Copy Markdown

Fixes #3099

Group routes now insert the missing separator when a group prefix and route path meet without /. Existing routes that already include the separator keep their current behavior.

Validation:

  • regression fails on pristine master with /v1posts and 404
  • focused group/route tests pass after the fix
  • git diff --check

Signed-off-by: Rayan-and-beyond <263488867+Rayan-and-beyond@users.noreply.github.com>
@aldas

aldas commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

group (prefix) does not necessarly need to mean segment in url that ends with slash.

@Rayan-and-beyond

Copy link
Copy Markdown
Author

That makes sense — the prefix can be an intentional partial path rather than a segment boundary, so inserting a slash would change valid routing semantics. I’ll close this rather than force that behavior. Thanks for clarifying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Group routes does not work without leading slash

2 participants