-
Notifications
You must be signed in to change notification settings - Fork 758
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
@SpringQueryMap doesn't work with String[] #318
Comments
You can implements the QueryMapEncoder interface.
And then declare a QueryMapEncoder bean in JavaConfig.
|
@rkovtiuk Does the provided workaround solve your issue? |
Hi @MissingNone, @OlgaMaciaszek, I also tried this variant:
But this also doesn't help. |
@rkovtiuk But I tried this is worked. config file:
FormQueryMapEncoder.java
version: Can you provide a demo for your issue? |
My configuration
version: |
@rkovtiuk I used your version for project And find out it. Beacause So adding the new configuration:
|
@MissingNone, this configuration works for me
But as I understood, this must work without this additional configuration. |
@rkovtiuk Yes,it's a bug. your issue is same as the issue #256.Issue OpenFeign/feign#1170 mentioned it. |
Closing in favour of OpenFeign/feign#1170. |
Describe the bug
@SpringQueryMap
doesn't support String[], but as@RequestParam
String[] is working.Sample
Fiegn client interface with two different API methods
ApiParams class
Expected Behaviour
Expected url for request with @SpringQueryMap:
https://url/incorrect?fields=name,surname
Generated url for request with @SpringQueryMap:
https://url/incorrect?fields=%5BLjava.lang.String%3B%405ed08faf&b=%5BLjava.lang.String%3B%4075386aa2
But when I'm trying to use String[], it works and generated url is as expected:
https://url/correct?fields=name,surname
The text was updated successfully, but these errors were encountered: