Matheus Cruz comments

Results 162 comments of


                                            Matheus Cruz

Show nicer error when a @ConfigMapping class exists in the deployment module and set to the runtime phase

Yes, I am not 100% comfortable with this approach (already implemented). Thank you for give one another solution. @gsmet Could you share the logic, just to understand how you achieve...

Show nicer error when a @ConfigMapping class exists in the deployment module and set to the runtime phase

Hi @gsmet and @radcortez I think it is ok now, could take a new review?

Show nicer error when a @ConfigMapping class exists in the deployment module and set to the runtime phase

> Looks like CI is passing! Could you squash the commits? Done!

Exclude uri from otel tracing

First draft pull request. Feel free to add any suggestions! cc: @brunobat

Allow to populate Spring @RestQuery Map<String, String> with request parameter values

cc: @cescoffier

Exclude uri from otel tracing

Sorry, for delay @brunobat. I have been very busy lately. I answered your comments!

Allow to populate Spring @RestQuery Map<String, String> with request parameter values

What if we add the same behavior but only for @RestQuery?

Exclude uri from otel tracing

I am out today, I will take a look tomorrow and let you know! I really did not remember On Sat, 8 Mar 2025 at 14:55 Giancarlo Calderón Cárdenas <...

Allow to populate Spring @RestQuery Map<String, String> with request parameter values

It would be great to support Map (It will help Spring users)... and to use a MultiValueMap (not the Spring) to support pure Jakarta users. At least one way will...

Allow to populate Spring @RestQuery Map<String, String> with request parameter values

See my [test](https://github.com/quarkusio/quarkus/blob/40b6c912fce6a7fa95c8f4712aef2e18faa658b3/extensions/spring-web/resteasy-reactive/tests/src/test/java/io/quarkus/spring/web/test/MapResourceWithExceptionTest.java#L23), I checked it and there is no handler. I remember that you added `List` with `@Separator`, but it is not related to that. I wanna create a...