Skip to content
This repository has been archived by the owner on Feb 4, 2022. It is now read-only.

Commit

Permalink
fix(connection): ensure socket options are applied to ssl sockets
Browse files Browse the repository at this point in the history
fixes NODE-1357
  • Loading branch information
mbroadst committed Mar 5, 2018
1 parent b479e8c commit e5ff927
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions lib/connection/connection.js
Original file line number Diff line number Diff line change
Expand Up @@ -573,7 +573,11 @@ function makeSSLConnection(self, _options) {
// We are done emit connect
self.emit('connect', self);
});

// Set the options for the connection

This comment has been minimized.

Copy link
@misshula

misshula Mar 5, 2018

👏 yes! this is now working. Thanks @mbroadst

connection.setKeepAlive(self.keepAlive, self.keepAliveInitialDelay);
connection.setTimeout(self.connectionTimeout);
connection.setNoDelay(self.noDelay);

return connection;
}
Expand Down

0 comments on commit e5ff927

Please sign in to comment.