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

Include full path of resource class in path parameter match #1199

Merged
merged 1 commit into from
Aug 2, 2022

Conversation

MikeEdgar
Copy link
Member

Fixes #1155

Signed-off-by: Michael Edgar [email protected]

@MikeEdgar MikeEdgar added the backport/2.x Issue pending backport to the 2.x series label Aug 1, 2022
@MikeEdgar MikeEdgar added this to the 3.1.0 milestone Aug 1, 2022
Copy link
Member

@phillip-kruger phillip-kruger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Did you test this in Quarkus ?

@MikeEdgar
Copy link
Member Author

LGTM. Did you test this in Quarkus ?

I haven't, but saw the same output that was in the original report (param missing) prior to making the change.

@MikeEdgar MikeEdgar merged commit 0a09b6d into smallrye:main Aug 2, 2022
@MikeEdgar MikeEdgar deleted the issue-1155 branch August 2, 2022 09:51
@MikeEdgar MikeEdgar removed the backport/2.x Issue pending backport to the 2.x series label Aug 9, 2022
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.

Missing path parameters for JAX-RS subresources declared in locator's @Path
2 participants