Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(node) Buffer.write(string, encoding, offset, length) is deprecated. Use write(string[, offset[, length]][, encoding]) instead. #364

Closed
manuel-di-iorio opened this issue Jan 7, 2016 · 5 comments
Closed

(node) Buffer.write(string, encoding, offset, length) is deprecated. Use write(string[, offset[, length]][, encoding]) instead. #364

manuel-di-iorio opened this issue Jan 7, 2016 · 5 comments

Comments

@manuel-di-iorio
Copy link

I downloaded the latest 1.0.7 version and this warning is still there. It appears randomly..
@zevero
Copy link

zevero commented May 24, 2016

Warning has been turned into an error! At least with latest node 6.2.0

Error: Buffer.write(string, encoding, offset[, length]) is no longer supported

This is solved by swapping arguments 2 and 3 on lines 61 and 62 in lib/multipart_parser.js

Please update!

@zevero
Copy link

zevero commented May 24, 2016

See
https://github.com/nodejs/node/blob/v6.2.0/doc/changelogs/CHANGELOG_V6.md#2016-04-26-version-600-current-jasnell

Previously deprecated Buffer APIs are removed #5048, #4594.

@redonkulus
Copy link

@egirshov @felixge any eta on this? as this package is relied on by superagent, this would block upgrading to latest node 6.x.

tmess567 added a commit to tmess567/heroku-buildpack-meteor that referenced this issue Sep 4, 2016
@tmess567
Trying out LTS node version 4.5.0
3f91128 
To allow multipart image handling. See node-formidable/formidable#364
@belain belain mentioned this issue Dec 28, 2016
Merged
mikeatlas added a commit to mikeatlas/meteor-tomi-upload-server that referenced this issue Jan 10, 2017
@mikeatlas
Use newer formidable - fixes Node 6.x compat
ee8ead1 
Hi - there's a newer (but not very new) update to Formidable which contains a fix to support Node 6.x:

node-formidable/formidable@f4a3e08
node-formidable/formidable#364
node-formidable/formidable#404
mikeatlas added a commit to mikeatlas/meteor-tomi-upload-server that referenced this issue Jan 10, 2017
@mikeatlas
update formidable for Node 6.x compat
b1bcf9a 
Hi - there's a newer (but not very new) update to Formidable which contains a fix to support Node 6.x:

node-formidable/formidable@f4a3e08
node-formidable/formidable#364
node-formidable/formidable#404
@mikeatlas mikeatlas mentioned this issue Jan 10, 2017
Merged
@felixge
Copy link
Collaborator

felixge commented Jan 11, 2017

@redonkulus please see #412

@kornelski
Copy link
Contributor

Fixed

@zevero zevero mentioned this issue Feb 27, 2017
Open
@hyiso hyiso mentioned this issue May 19, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@felixge @kornelski @redonkulus @zevero @manuel-di-iorio