Support for HTTP GET map parameters

[kwondh5217]

Motivation:

This pull request addresses #6058

Currently, @Param cannot be mapped to a Map, but this change enables that functionality, allowing query parameters to be handled as a Map.

Modifications:

• Introduced logic in AnnotatedValueResolver to detect when a parameter is of type Map and the @Param annotation has an unspecified value.
  • Future enhancements may include supporting the mapping of specific values to a Map when the @Param value is explicitly specified.
• Added a new method ofQueryParamMap to handle the creation of an AnnotatedValueResolver for mapping all query parameters into a Map.
  • Query parameters are collected into a Map using the stream-based collector.
• Updated the existing logic to ensure this behavior only applies when the @Param value is unspecified and the parameter type is Map.

Result:

• Closes Support for HTTP GET map parameters? #6058
• Users can now define a method parameter annotated with @Param as Map<String, Object> to handle all query parameters.

[github-actions]

🔍 Build Scan® (commit: f3ba5a2)

Job name	Status	Build Scan®
build-ubicloud-standard-16-jdk-8	✅	https://ge.armeria.dev/s/pncnbudgrtywa
build-ubicloud-standard-16-jdk-21-snapshot-blockhound	✅	https://ge.armeria.dev/s/tdy4gcysqxwks
build-ubicloud-standard-16-jdk-17-min-java-17-coverage	✅	https://ge.armeria.dev/s/dfomdckvyl6hs
build-ubicloud-standard-16-jdk-17-min-java-11	✅	https://ge.armeria.dev/s/koskhl7sqewzu
build-ubicloud-standard-16-jdk-17-leak	✅	https://ge.armeria.dev/s/hupdx7mgw27ry
build-ubicloud-standard-16-jdk-11	✅	https://ge.armeria.dev/s/gdalxrcr6etlu
build-macos-latest-jdk-21	✅	https://ge.armeria.dev/s/4kjtrovaxxahm

[minwoox]

HI, @kwondh5217, thanks for the PR. 😉
Would you add an e2e test that verifies if this change works?

GET http://127.0.0.1:xxx/map?a=b&c=d

@Get("/map")
public HttpResponse map(@Params Map<String, Object> map) {...}

[kwondh5217]

@minwoox Thanks for your comment!
I've added a test in AnnotatedServiceTest to verify that this change works.
Please take a look when you have time! 🙇

[minwoox]

Basically looks good. Left just nits. 👍

[minwoox]

It seems like this line of code is not invoked. Would you explain why you've added this?

[kwondh5217]

@minwoox Thank you for pointing this out! 🙇
This line was added by mistake and isn't necessary. I've removed it now. Appreciate your attention to detail!

[minwoox]

How about raising an exception if the value is specified?

if (Map.class.isAssignableFrom(type)) {
    if (DefaultValues.isSpecified(param.value())) {
        // throw an exception...
    }
    return ofQueryParamMap(name, annotatedElement, typeElement, type, description);
}

[kwondh5217]

@minwoox Thank you for the suggestion! 🙇
I've updated the code to raise an exception when a value is specified for a Map parameter and added the test cases. Please take a look when you have time.