security: New setting for Socket.IO maxHttpBufferSize

This commit is contained in:
John McLear 2021-02-14 19:04:29 +00:00 committed by Richard Hansen
parent ed93ef5636
commit b7e88cb904
5 changed files with 39 additions and 4 deletions

View file

@ -74,7 +74,7 @@ exports.expressCreateServer = (hookName, args, cb) => {
* https://github.com/socketio/socket.io/issues/2276#issuecomment-147184662 (not totally true, actually, see above)
*/
cookie: false,
maxHttpBufferSize: 10E3,
maxHttpBufferSize: settings.socketIo.maxHttpBufferSize,
});
io.on('connect', (socket) => {

View file

@ -104,6 +104,18 @@ exports.ssl = false;
**/
exports.socketTransportProtocols = ['xhr-polling', 'jsonp-polling', 'htmlfile'];
exports.socketIo = {
/**
* Maximum permitted client message size (in bytes).
*
* All messages from clients that are larger than this will be rejected. Large values make it
* possible to paste large amounts of text, and plugins may require a larger value to work
* properly, but increasing the value increases susceptibility to denial of service attacks
* (malicious clients can exhaust memory).
*/
maxHttpBufferSize: 10000,
};
/*
* The Type of the database
*/