tls: set servername on client side too

jasnell · MylesBorins · commit 3997617869d0 · 2018-01-10T01:42:19.000-05:00
Backport-PR-URL: #18050 PR-URL: #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
@@ -627,7 +627,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
`@@ -627,7 +627,7 @@ TLSSocket.prototype._finishInit = function() {`
`627`	`627`	`this.alpnProtocol = this._handle.getALPNNegotiatedProtocol();`
`628`	`628`	`}`
`629`	`629`
`630`		`- if (process.features.tls_sni && this._tlsOptions.isServer) {`
	`630`	`+ if (process.features.tls_sni) {`
`631`	`631`	`this.servername = this._handle.getServername();`
`632`	`632`	`}`
`633`	`633`