net/http: memory leak in http.Transport

[gopherbot]

by pkulak:

What steps will reproduce the problem?

getIdleConnCh() is called for every round trip (called by getConn, which is called by
RoundTrip), regardless of weather keepAlives are on. In my case, with an app that's
connecting to hundreds of thousands of different hosts, that idleConnCh map just keeps
filling up and filling up forever.


Which operating system are you using?

OSX and Ubuntu


Which version are you using?  (run 'go version')

"go version go1.1 darwin/amd64"
"go version go1.1.1 linux/amd64"

[bradfitz]

Comment 1:

Owner changed to @bradfitz.

Status changed to Accepted.

[bradfitz]

Comment 2:

Thanks! Fix out for review: https://golang.org/cl/10747044

Status changed to Started.

[bradfitz]

Comment 3:

This issue was closed by revision 46161cd.

Status changed to Fixed.