public-offering
/
pad.pub0.org
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

socketio: Use Error objects for socket.io connection errors 

socket.io expects Error objects, otherwise it won't propagate the
message to the client.

Also do some cleanup.
pull/4269/head
Richard Hansen 2020-09-14 00:49:16 -04:00 committed by John McLear
parent 0a836ced29
commit 259b8d891d
1 changed files with 13 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/node/hooks/express/socketio.js
View File

@ -49,24 +49,24 @@ exports.expressCreateServer = function (hook_name, args, cb) {
// modify, or create any pad (unless the pad is password protected or an HTTP API session is
// required).
var cookieParserFn = cookieParser(webaccess.secret, {});
io.use(function(socket, accept) {
io.use((socket, next) => {
var data = socket.request;
if (!data.headers.cookie && settings.loadTest) return accept(null, true);
cookieParserFn(data, {}, function(err) {
if (err) {
console.error(err);
accept("Couldn't parse request cookies.", false);
return;
if (!data.headers.cookie && settings.loadTest) {
console.warn('bypassing socket.io authentication check due to settings.loadTest');
return next(null, true);
}
data.sessionID = data.signedCookies.express_sid;
if (!data.sessionID) return accept('Signed express_sid cookie is required', false);
args.app.sessionStore.get(data.sessionID, function(err, session) {
if (err || !session) return accept('Bad session / session has expired', false);
const fail = (msg) => { return next(new Error(msg), false); };
cookieParserFn(data, {}, function(err) {
if (err) return fail('access denied: unable to parse express_sid cookie');
const expressSid = data.signedCookies.express_sid;
if (!expressSid) return fail ('access denied: signed express_sid cookie is required');
args.app.sessionStore.get(expressSid, (err, session) => {
if (err || !session) return fail('access denied: bad session or session has expired');
data.session = new sessionModule.Session(data, session);
if (settings.requireAuthentication && data.session.user == null) {
return accept('Authentication required', false);
return fail('access denied: authentication required');
}
accept(null, true);
next(null, true);
});
});
});