Skip to content

Commit

Permalink
tls: set servername on client side too
Browse files Browse the repository at this point in the history
Backport-PR-URL: #18050
Backport-PR-URL: #20456
PR-URL: #17935
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Sebastiaan Deckers <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
  • Loading branch information
jasnell authored and MylesBorins committed May 2, 2018
1 parent 1a24fec commit 95c1e2d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/_tls_wrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -628,7 +628,7 @@ TLSSocket.prototype._finishInit = function() {
this.alpnProtocol = this.ssl.getALPNNegotiatedProtocol();
}

if (process.features.tls_sni && this._tlsOptions.isServer) {
if (process.features.tls_sni) {
this.servername = this._handle.getServername();
}

Expand Down

0 comments on commit 95c1e2d

Please sign in to comment.