updating code review changes

Author: adrian05-ms

components/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/RedirectHandler.java

@@ -179,7 +179,7 @@ Request getRedirect(
 
                 final Request followup =
                         shouldRedirect
-                                ? getRedirect(request, response, redirectOption, chain)
+                                ? getRedirect(request, response, redirectOption)
                                 : null;
                 if (followup != null) {
                     response.close();

components/http/okHttp/src/test/java/com/microsoft/kiota/http/middleware/RedirectHandlerTests.java

@@ -85,7 +85,7 @@ void crossHostRedirectStripsAuthHeaders() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption();
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
         assertNotNull(result);
         assertEquals("evil.attacker.com", result.url().host());
         assertNull(result.header("Authorization")); // stripped (good)
@@ -145,7 +145,7 @@ void sameHostRedirectKeepsAllHeaders() throws Exception {
         RedirectHandlerOption option = new RedirectHandlerOption();
         Request result =
                 new RedirectHandler(option, proxySelector)
-                        .getRedirect(original, redirect, option, chain);
+                        .getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("trusted.example.com", result.url().host());
@@ -182,7 +182,7 @@ void crossHostRedirectWithProxyKeepsProxyAuth() throws Exception {
         RedirectHandlerOption option = new RedirectHandlerOption();
         Request result =
                 new RedirectHandler(option, proxySelector)
-                        .getRedirect(original, redirect, option, chain);
+                        .getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("other.example.com", result.url().host());
@@ -220,7 +220,7 @@ void crossHostRedirectWithDirectProxyStripsProxyAuth() throws Exception {
         RedirectHandlerOption option = new RedirectHandlerOption();
         Request result =
                 new RedirectHandler(option, proxySelector)
-                        .getRedirect(original, redirect, option, chain);
+                        .getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("other.example.com", result.url().host());
@@ -256,7 +256,7 @@ void schemeChangeStripsAuthHeaders() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption();
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("trusted.example.com", result.url().host());
@@ -288,7 +288,7 @@ void redirectWithDifferentPortRemovesAuthAndCookie() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption();
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("example.org", result.url().host());
@@ -319,7 +319,7 @@ void redirectWithSamePortKeepsAuthAndCookie() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption();
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("example.org", result.url().host());
@@ -356,7 +356,7 @@ void customScrubberIsUsed() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption(5, null, customScrubber);
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("other.example.com", result.url().host());
@@ -398,7 +398,7 @@ void customScrubberRemovesCustomHeaders() throws Exception {
         Interceptor.Chain chain = mock(Interceptor.Chain.class);
 
         RedirectHandlerOption option = new RedirectHandlerOption(5, null, customScrubber);
-        Request result = new RedirectHandler().getRedirect(original, redirect, option, chain);
+        Request result = new RedirectHandler().getRedirect(original, redirect, option);
 
         assertNotNull(result);
         assertEquals("other.example.com", result.url().host());