-
Notifications
You must be signed in to change notification settings - Fork 58
Open
Description
for example here:
Lines 32 to 37 in 01e61e9
options = options || {}; | |
options.keepAlive = options.keepAlive !== false; | |
// default is keep-alive and 15s free socket timeout | |
if (options.freeSocketTimeout === undefined) { | |
options.freeSocketTimeout = 15000; | |
} |
If the options argument is defined it should be cloned before having it's props mutated.
I'm happy to submit a PR, just wanted to check that the author agrees with the suggestion
Metadata
Metadata
Assignees
Labels
No labels