Skip to content

Commit 5746c34

Browse files
committed
fix: check for port changes on redirect
1 parent cf142ea commit 5746c34

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,11 @@ void scrubHeaders(
7070
Objects.requireNonNull(originalUrl, "parameter originalUrl cannot be null");
7171
Objects.requireNonNull(newUrl, "parameter newUrl cannot be null");
7272

73-
// Remove Authorization and Cookie headers if the request's scheme or host changes
73+
// Remove Authorization and Cookie headers if the request's scheme, host, or port changes
7474
boolean isDifferentHostOrScheme =
7575
!newUrl.host().equalsIgnoreCase(originalUrl.host())
76-
|| !newUrl.scheme().equalsIgnoreCase(originalUrl.scheme());
76+
|| !newUrl.scheme().equalsIgnoreCase(originalUrl.scheme())
77+
|| newUrl.port() != originalUrl.port();
7778
if (isDifferentHostOrScheme) {
7879
requestBuilder.removeHeader("Authorization");
7980
requestBuilder.removeHeader("Cookie");

0 commit comments

Comments
 (0)