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

jax-rs - exception mapper - Unable to extract parameter doesn't show the value of the parameter, just {1} #11252

Closed
rsvoboda opened this issue Aug 6, 2020 · 3 comments
Labels
area/resteasy-classic kind/bug Something isn't working triage/out-of-date This issue/PR is no longer valid or relevant
Milestone

Comments

@rsvoboda
Copy link
Member

rsvoboda commented Aug 6, 2020

Quarkus master / Quarkus 1.7.0.CR2 - Unable to extract parameter doesn't show the value of the parameter, just {1} for case of exception in conversion of @QueryParam to the expected type - String to Integer in my case

Regression comparing to 1.3.4.Final and 1.6.1.Final, found it when experimenting with https://github.com/gastaldi/resteasy-xss example
It's REASTEasy issue for sure, I have possible fix for it.
Want to track it here too so as it's regression and is connected to #7248

What I see: RESTEASY003870: Unable to extract parameter from http request: javax.ws.rs.QueryParam("paging") value is {1} for public java.lang.Integer org.acme.UsersResource.page(java.lang.Integer)

Expectation: RESTEASY003870: Unable to extract parameter from http request: javax.ws.rs.QueryParam("paging") value is 'asd' for public java.lang.Integer org.acme.UsersResource.page(java.lang.Integer)

Request: http://localhost:8080/users?paging=asd

@rsvoboda rsvoboda added kind/bug Something isn't working area/resteasy-classic labels Aug 6, 2020
@rsvoboda
Copy link
Member Author

rsvoboda commented Aug 6, 2020

RESTEasy issue: https://issues.redhat.com/browse/RESTEASY-2674
Possible fix: resteasy/resteasy#2490

@geoand
Copy link
Contributor

geoand commented Jul 27, 2021

It appears from the RESTEasy version where this was fixed that it is no longer an issue

@geoand geoand closed this as completed Jul 27, 2021
@geoand geoand added the triage/out-of-date This issue/PR is no longer valid or relevant label Jul 27, 2021
@rsvoboda
Copy link
Member Author

+1, the issue is fixed now, checked with Quarkus 2.1.0.Final

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/resteasy-classic kind/bug Something isn't working triage/out-of-date This issue/PR is no longer valid or relevant
Projects
None yet
Development

No branches or pull requests

2 participants