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

FIXED: use Buffer.from(string) instead of Buffer(string)

Open warpdesign opened this issue 6 years ago • 6 comments

Fixes Node.js 10 deprecated warning, see: https://nodejs.org/api/buffer.html#buffer_new_buffer_string_encoding

warpdesign avatar Nov 20 '18 23:11 warpdesign

@mscdex are you able to merge this?

jasonwilliams avatar Nov 25 '21 07:11 jasonwilliams

Someone please merge this... I really need to use this library and this error is preventing me from doing so.

mgrist avatar Oct 24 '22 20:10 mgrist

Someone please merge this... I really need to use this library and this error is preventing me from doing so.

I’m pretty sure this library is dead, https://github.com/mscdex/node-ftp/issues/277

jasonwilliams avatar Oct 26 '22 09:10 jasonwilliams

Someone please merge this... I really need to use this library and this error is preventing me from doing so.

I’m pretty sure this library is dead, #277

Yeah, I thought so. Thanks for the heads up. Ended up using a different library.

mgrist avatar Oct 26 '22 16:10 mgrist

@mgrist as a matter of interest, which library are you using now?

pedro-w avatar Oct 27 '22 07:10 pedro-w

@mgrist as a matter of interest, which library are you using now?

I ended up using basic-ftp. It was really easy to use and seems to have regular updates to the repository. I mainly needed an FTP library that could work with streams. If you have any questions just let me know.

mgrist avatar Oct 27 '22 15:10 mgrist