node-ftp icon indicating copy to clipboard operation
node-ftp copied to clipboard

Consistency in passive socket's variable name

Open gbourel opened this issue 8 years ago • 0 comments

When reading current lib/connection.js implementation ( commit 7dff82f ) I noticed that the passive socket reference is mainly written as _pasvSock but sometimes written _pasvSocket (with the trailing characters 'et' ): Defined at L60 this._pasvSock = undefined; Used as _pasvSocket in both _pasvConnect() L903, _send() L1034 methods. But again written as _pasvSock in _reset() L1053 method.

Did I miss the usage of the two different references with different meaning or may it be a typo ?

gbourel avatar Mar 14 '17 17:03 gbourel