Skip to content

Commit 9956445

Browse files
oliversalzburgdarrachequesne
authored andcommitted
[fix] Replace deprecated Buffer usage (#565)
The `Buffer` constructor has been deprecated in favor of safer alternatives. See https://nodejs.org/en/docs/guides/buffer-constructor-deprecation/
1 parent e081616 commit 9956445

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ var server = engine.listen(80);
2020

2121
server.on('connection', function(socket){
2222
socket.send('utf 8 string');
23-
socket.send(new Buffer([0, 1, 2, 3, 4, 5])); // binary data
23+
socket.send(Buffer.from([0, 1, 2, 3, 4, 5])); // binary data
2424
});
2525
```
2626

lib/server.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -359,8 +359,7 @@ Server.prototype.handleUpgrade = function (req, socket, upgradeHead) {
359359
return;
360360
}
361361

362-
var head = new Buffer(upgradeHead.length); // eslint-disable-line node/no-deprecated-api
363-
upgradeHead.copy(head);
362+
var head = Buffer.from(upgradeHead); // eslint-disable-line node/no-deprecated-api
364363
upgradeHead = null;
365364

366365
// delegate to ws

0 commit comments

Comments
 (0)