RedirectHandler appears to be redundant for default configuration

[p-m-j]

followRedirects(false) isn't called on the default OkHttpClient builder and so redirects are followed upstream from kiota, this makes RedirectHandler a little confusing.

Forgive me If this is known and intentional, it tripped me up for a while.

[baywet]

Hi @p-m-j
Thank you for using kiota and for reaching out.

I think this is an oversight on our end, we meant to align java with the other languages, and disable the default redirection following behaviour.

Is this something you'd like to submit a pull request for provided some guidance?

[p-m-j]

@baywet I'm happy to submit a PR (nice to see that the CLA setup looks pretty painless 👍).

Guide away!

[baywet]

The factory that sets up the default http client is here.

kiota-java/components/http/okHttp/src/main/java/com/microsoft/kiota/http/KiotaClientFactory.java

Line 59 in c4684ab

final OkHttpClient.Builder builder =

And you can add an additional unit test here to prevent future regressions on the setup

kiota-java/components/http/okHttp/src/test/java/com/microsoft/kiota/http/KiotaClientFactoryTest.java

Line 33 in c4684ab

void testCreatesDefaultInterceptors() throws IOException {

Optional, the redirect handler does not have any unit test 😱 But if you also want to take on that extra work, you could add a new file here
https://github.com/microsoft/kiota-java/tree/main/components/http/okHttp/src/test/java/com/microsoft/kiota/http/middleware

Let me know if you have any additional comments or questions.

[p-m-j]

Apologies for the turnaround time @baywet, the folks that pay the bills have kept me pretty busy.