net/http: drop unused "broken" field from persistConn

Change-Id: Ic65cf98c090c73299b5e88e642e91139315d8e52
Reviewed-on: https://go-review.googlesource.com/c/go/+/722221
Auto-Submit: Damien Neil <dneil@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Nicholas Husin <husin@google.com>
Reviewed-by: Nicholas Husin <nsh@golang.org>
This commit is contained in:
Damien Neil 2025-11-19 14:35:20 -08:00 committed by Gopher Robot
parent 4b740af56a
commit ca37d24e0b

View file

@ -2110,7 +2110,6 @@ type persistConn struct {
numExpectedResponses int numExpectedResponses int
closed error // set non-nil when conn is closed, before closech is closed closed error // set non-nil when conn is closed, before closech is closed
canceledErr error // set non-nil if conn is canceled canceledErr error // set non-nil if conn is canceled
broken bool // an error has happened on this connection; marked broken so it's not reused.
reused bool // whether conn has had successful request/response and is being reused. reused bool // whether conn has had successful request/response and is being reused.
// mutateHeaderFunc is an optional func to modify extra // mutateHeaderFunc is an optional func to modify extra
// headers on each outbound request before it's written. (the // headers on each outbound request before it's written. (the
@ -2925,7 +2924,6 @@ func (pc *persistConn) closeLocked(err error) {
if err == nil { if err == nil {
panic("nil error") panic("nil error")
} }
pc.broken = true
if pc.closed == nil { if pc.closed == nil {
pc.closed = err pc.closed = err
pc.t.decConnsPerHost(pc.cacheKey) pc.t.decConnsPerHost(pc.cacheKey)