tls: set servername on client side too

jasnell · jasnell · commit b25b1efa0a68 · 2018-01-03T11:29:39.000-08:00
PR-URL: nodejs#17935 Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
@@ -621,7 +621,7 @@ TLSSocket.prototype._finishInit = function() {
     this.alpnProtocol = this._handle.getALPNNegotiatedProtocol();
   }
 
-  if (process.features.tls_sni && this._tlsOptions.isServer) {
+  if (process.features.tls_sni) {
     this.servername = this._handle.getServername();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -621,7 +621,7 @@ TLSSocket.prototype._finishInit = function() {`
`621`	`621`	`this.alpnProtocol = this._handle.getALPNNegotiatedProtocol();`
`622`	`622`	`}`
`623`	`623`
`624`		`- if (process.features.tls_sni && this._tlsOptions.isServer) {`
	`624`	`+ if (process.features.tls_sni) {`
`625`	`625`	`this.servername = this._handle.getServername();`
`626`	`626`	`}`
`627`	`627`