Deprecation Warning

[muz3]

hi team, I received the following warning on my app build, after tracing it down, I found out mqtt-packet still uses Buffer() here "https://github.com/mqttjs/mqtt-packet/blob/master/constants.js#L131". Can we please get it fixed? thanks,

"(node:8008) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead."

[mcollina]

would you mind sending a PR? Il giorno sab 27 lug 2019 alle 00:06 rayw <notifications@github.com> ha scritto:

…

hi team, I received the following warning on my app build, after tracing it down, I found out mqtt-packet still uses Buffer() here " https://github.com/mqttjs/mqtt-packet/blob/master/constants.js#L131". Can we please get it fixed? thanks, "(node:8008) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead." — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#63?email_source=notifications&email_token=AAAMXY25TJ4ODZ6KS7RD7C3QBNYNRA5CNFSM4IHHP3HKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HBZ23TQ>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAMXY2IPPQ3S57GUEAQUFDQBNYNRANCNFSM4IHHP3HA> .

[muz3]

done. PR is submitted here: #64

changed from 'new Buffer(1)' to Buffer.alloc(1). no more deprecated warning, and passed npm test.