Connection: Add option to specify keep-alive behaviour on connect.

mscdex / node-ftp

An FTP client module for node.js

MIT License

1.13k stars 246 forks source link

Connection: Add option to specify keep-alive behaviour on connect. #192

Open peterkooijmans opened 7 years ago

peterkooijmans commented 7 years ago

At one of our customers with a DELL Sonic firewall, the FTP connection would be dropped when keep-alives are enabled. This pull request adds an option to specify the keep-alive behaviour of the FTP connection and defaults to the safe option of disabling it.

rogierschouten commented 6 years ago

Is there any chance of this PR to get merged?

kelvinau commented 6 years ago

This should be merged. It's unable to make the connection when setSocketAlive is true for GoDaddy FTP too.