Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jooby-apt: must add leading slash on path pattern #3468

Closed
surisuku opened this issue Jul 9, 2024 · 0 comments
Closed

jooby-apt: must add leading slash on path pattern #3468

surisuku opened this issue Jul 9, 2024 · 0 comments
Milestone

Comments

@surisuku
Copy link

surisuku commented Jul 9, 2024

The changes made to the mvc processing on 3.2.0 seem to have changed the path on some routes which didn't have an explicit '/' character on the specified path.

Taking the following code:

public class Test {
    @GET("test")
    public Object test() {}
}

This would produce a route for "/test" on version 3.1.4, but would produce "test" on 3.2.0.

From what I can see this change is not documented on the release notes for the version.

@jknack jknack added this to the 3.2.5 milestone Jul 10, 2024
@jknack jknack changed the title Undocumented behaviour change on 3.2.x jooby-apt: must add leading slash on path Jul 10, 2024
@jknack jknack changed the title jooby-apt: must add leading slash on path jooby-apt: must add leading slash on path pattern Jul 10, 2024
@jknack jknack closed this as completed in b3f6ffc Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants