RestClient and request filter: the http request does not want pass through the filter

[Gianfelice Catini]

Hi,

I'm trying to develop a very simple filter/interceptor that should add a header to the http requests made by a @RestClient.

My Quarkus app ignores the filter like it doesnt exist: the http requests complete successfully, but they dont go through the filter.

I reproduced the described behaviour here:

https://github.com/dbl81/test-filter.git

It's a very basic code with a @RestClient and a request filter.

Can anyone help me figure out what i am doing wrong?

Thank you.

[Ken Finnigan]

@Provider annotated filters are only applied to JAX-RS Server requests, not Client ones.

To have a filter added for REST Client you need to add @RegisterProvider(LoggingFilter.class) onto your RestClient interface

--
You received this message because you are subscribed to the Google Groups "Quarkus Development mailing list" group.
To unsubscribe from this group and stop receiving emails from it, send an email to quarkus-dev...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/quarkus-dev/148ff77e-1fc6-4267-836a-87ac3a65f4bdn%40googlegroups.com.

[Gianfelice Catini]

Ken, thank you for your quick answer.

I tried to put @RegisterProvider(LoggingFilter.class) in my Rest Client interface (I also removed the @Provider annotation on the LoggingFilter.class), but it didnt solve my problem.

I pushed the new code into the repository.

[Ken Finnigan]

Oops, didn't notice it before, but the filter implements the wrong interface. It needs to implement ClientRequestFilter or ClientResponseFilter. ContainerRequestFilter only works on server requests

[Gianfelice Catini]

I tried to implement the right interfaces (I implemented the ClientRequestFilter and the ClientRequestResponse both), but the client requests/responses dont go through the filter.

I pushed the code again.

Thank you for your patience and your answers.

[Gianfelice Catini]

I found the mistake: it was an error on my test method (bad usage of the rest client).

Ken, thank you for your support: I followed your hints and now it works well.

[Ken Finnigan]

Great to hear

To view this discussion on the web visit https://groups.google.com/d/msgid/quarkus-dev/0858476c-e378-4390-a12f-d1c9a727ce64n%40googlegroups.com.