Compare commits

...

2 Commits

Author SHA1 Message Date
masterZSH 32068fbad6
Merge c19c24aa37 into 70bf50955e 2024-06-19 17:42:48 +10:00
zsh c19c24aa37 Fix Manually passed `Cookie` header overrides `http.CookieJar` cookies 2020-06-08 16:16:12 +08:00
1 changed files with 7 additions and 7 deletions

View File

@ -196,13 +196,6 @@ func (d *Dialer) DialContext(ctx context.Context, urlStr string, requestHeader h
} }
req = req.WithContext(ctx) req = req.WithContext(ctx)
// Set the cookies present in the cookie jar of the dialer
if d.Jar != nil {
for _, cookie := range d.Jar.Cookies(u) {
req.AddCookie(cookie)
}
}
// Set the request headers using the capitalization for names and values in // Set the request headers using the capitalization for names and values in
// RFC examples. Although the capitalization shouldn't matter, there are // RFC examples. Although the capitalization shouldn't matter, there are
// servers that depend on it. The Header.Set method is not used because the // servers that depend on it. The Header.Set method is not used because the
@ -234,6 +227,13 @@ func (d *Dialer) DialContext(ctx context.Context, urlStr string, requestHeader h
} }
} }
// Set the cookies present in the cookie jar of the dialer
if d.Jar != nil {
for _, cookie := range d.Jar.Cookies(u) {
req.AddCookie(cookie)
}
}
if d.EnableCompression { if d.EnableCompression {
req.Header["Sec-WebSocket-Extensions"] = []string{"permessage-deflate; server_no_context_takeover; client_no_context_takeover"} req.Header["Sec-WebSocket-Extensions"] = []string{"permessage-deflate; server_no_context_takeover; client_no_context_takeover"}
} }