Skip to content

Commit 572663f

Browse files
skenqbxjasnell
authored andcommitted
https: don't overwrite servername option
PR-URL: nodejs/node-v0.x-archive#9368 Reviewed-By: jasnell - James M Snell <[email protected]>
1 parent 75c84b2 commit 572663f

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

lib/http.js

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1283,11 +1283,13 @@ Agent.prototype.createSocket = function(name, host, port, localAddress, req) {
12831283
options.host = host;
12841284
options.localAddress = localAddress;
12851285

1286-
options.servername = host;
1287-
if (req) {
1288-
var hostHeader = req.getHeader('host');
1289-
if (hostHeader) {
1290-
options.servername = hostHeader.replace(/:.*$/, '');
1286+
if (!options.servername) {
1287+
options.servername = host;
1288+
if (req) {
1289+
var hostHeader = req.getHeader('host');
1290+
if (hostHeader) {
1291+
options.servername = hostHeader.replace(/:.*$/, '');
1292+
}
12911293
}
12921294
}
12931295

0 commit comments

Comments
 (0)