crypto/tls: Conn.ConnectionState should set state.ServerName from c.s…

…erverName unconditionally - handshakeComplete is not necessary for SNI (golang#15571)

Moves the state.ServerName assignment to outside the if statement that checks for handshakeComplete.

src/crypto/tls/conn.go

@@ -1209,6 +1209,7 @@ func (c *Conn) ConnectionState() ConnectionState {
 
 	var state ConnectionState
 	state.HandshakeComplete = c.handshakeComplete
+	state.ServerName = c.serverName
 	if c.handshakeComplete {
 		state.Version = c.vers
 		state.NegotiatedProtocol = c.clientProtocol
@@ -1217,7 +1218,6 @@ func (c *Conn) ConnectionState() ConnectionState {
 		state.CipherSuite = c.cipherSuite
 		state.PeerCertificates = c.peerCertificates
 		state.VerifiedChains = c.verifiedChains
-		state.ServerName = c.serverName
 		state.SignedCertificateTimestamps = c.scts
 		state.OCSPResponse = c.ocspResponse
 		if !c.didResume {