]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.8] crypto/tls: make Config.Clone also clone the GetClientCertific...
authorMike Danese <mikedanese@google.com>
Wed, 1 Mar 2017 18:43:57 +0000 (10:43 -0800)
committerBrad Fitzpatrick <bradfitz@golang.org>
Wed, 8 Mar 2017 21:19:55 +0000 (21:19 +0000)
commitea6781bcd0fd8a0b59d5d617cb73516bd4f4ad12
tree726cdecf2f598312f6494abc29b2c6fe9572815e
parent2327d696c12155c6b94b3e1e74cbf059d2e07f37
[release-branch.go1.8] crypto/tls: make Config.Clone also clone the GetClientCertificate field

Using GetClientCertificate with the http client is currently completely
broken because inside the transport we clone the tls.Config and pass it
off to the tls.Client. Since tls.Config.Clone() does not pass forward
the GetClientCertificate field, GetClientCertificate is ignored in this
context.

Fixes #19264

Change-Id: Ie214f9f0039ac7c3a2dab8ffd14d30668bdb4c71
Signed-off-by: Mike Danese <mikedanese@google.com>
Reviewed-on: https://go-review.googlesource.com/37541
Reviewed-by: Filippo Valsorda <hi@filippo.io>
Reviewed-by: Adam Langley <agl@golang.org>
Run-TryBot: Adam Langley <agl@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
(cherry picked from commit 87649d32ad16a9a0b7bd5dbd1c124b2032a270f1)
Reviewed-on: https://go-review.googlesource.com/37946
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Tom Bergan <tombergan@google.com>
src/crypto/tls/common.go
src/crypto/tls/tls_test.go