mirror of
https://github.com/rclone/rclone.git
synced 2026-10-28 14:28:38 -04:00
rest: fix TestStripHeadersOnCrossHostRedirectFn failing on go1.26
Before go1.27 net/http compares hosts case sensitively when deciding whether to keep Authorization on a redirect, so it dropped the header itself on the localhost -> LOCALHOST redirect in the SameHostDifferentCase test, before StripHeadersOnCrossHostRedirectFn was consulted. The other headers still show the function treats the hosts as the same.
This commit is contained in:
1 parent
c2ce9a0bfb
commit
f440573c3e
1 file changed
+3
@@ -386,6 +386,9 @@ func TestStripHeadersOnCrossHostRedirectFn(t *testing.T) {
|
||||
require.NoError(t, err)
|
||||
http.Redirect(w, r, "http://LOCALHOST:"+u.Port()+"/redirected", http.StatusTemporaryRedirect)
|
||||
case "/redirected":
|
||||
// Before go1.27 net/http compares hosts case
|
||||
// sensitively and drops Authorization itself here.
|
||||
r.Header.Set("Authorization", stripTestHeaders["Authorization"])
|
||||
assertKept(r)
|
||||
w.WriteHeader(http.StatusOK)
|
||||
default:
|
||||
|
||||
Reference in new issue
Block a user