fix rest client reactive exception handling guide

[felipewind]

I believe that:

• The implementation of ResponseExceptionMapper should be done by a class and not an interface.
• The method toThrowable should be public, or I receive this error: Cannot reduce the visibility of the inherited method from ResponseExceptionMapper<RuntimeException>

Current version

public interface MyResponseExceptionMapper implements ResponseExceptionMapper<RuntimeException> {

    RuntimeException toThrowable(Response response) {
        if (response.getStatus() == 500) {
            throw new RuntimeException("The remote service responded with HTTP 500");
        }
        return null;
    }
}

Proposed version

public class MyResponseExceptionMapper implements ResponseExceptionMapper<RuntimeException> {

    @Override
    public RuntimeException toThrowable(Response response) {
        if (response.getStatus() == 500) {
            throw new RuntimeException("The remote service responded with HTTP 500");
        }
        return null;
    }
}

[gsmet]

Hi @felipewind .

Thanks for this but changes to the guides need to be done there: https://github.com/quarkusio/quarkus/tree/main/docs/src/main/asciidoc .

Once it's in the main repo, I'll merge your PR though as fixing old branches is a good idea.

[felipewind]

Hello @gsmet .

Sorry about the mistake.

I sent the PR as you asked:

• Fix Guide Rest Client Reactive Exception Handling quarkus#25714

Please tell me if I must change anything.

Thanks

[gastaldi]

Thank you! Closing this now