From: Damien Neil <dneil@google.com>
Date: Wed, 19 Nov 2025 22:35:20 +0000 (-0800)
Subject: net/http: drop unused "broken" field from persistConn
X-Git-Tag: go1.26rc1~226
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=ca37d24e0b9369b8086959df5bc230b38bf98636;p=gostls13.git

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>
---

diff --git a/src/net/http/transport.go b/src/net/http/transport.go
index a560765d33..4e6b07f34d 100644
--- a/src/net/http/transport.go
+++ b/src/net/http/transport.go
@@ -2110,7 +2110,6 @@ type persistConn struct {
 	numExpectedResponses int
 	closed               error // set non-nil when conn is closed, before closech is closed
 	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.
 	// mutateHeaderFunc is an optional func to modify extra
 	// headers on each outbound request before it's written. (the
@@ -2925,7 +2924,6 @@ func (pc *persistConn) closeLocked(err error) {
 	if err == nil {
 		panic("nil error")
 	}
-	pc.broken = true
 	if pc.closed == nil {
 		pc.closed = err
 		pc.t.decConnsPerHost(pc.cacheKey)