fix: #9231, fix redis pubsub connection

regression from fdfbc90255

src/database/redis.js

@@ -103,10 +103,10 @@ redisModule.info = async function (cxn) {
 	return redisData;
 };
 
-redisModule.socketAdapter = function () {
+redisModule.socketAdapter = async function () {
 	const redisAdapter = require('socket.io-redis');
-	const pub = connection.connect(nconf.get('redis'));
-	const sub = connection.connect(nconf.get('redis'));
+	const pub = await connection.connect(nconf.get('redis'));
+	const sub = await connection.connect(nconf.get('redis'));
 	return redisAdapter({
 		key: 'db:' + nconf.get('redis:database') + ':adapter_key',
 		pubClient: pub,

src/database/redis/pubsub.js

@@ -1,31 +1,35 @@
 'use strict';
 
-var nconf = require('nconf');
-var util = require('util');
-var winston = require('winston');
-var EventEmitter = require('events').EventEmitter;
+const nconf = require('nconf');
+const util = require('util');
+const winston = require('winston');
+const EventEmitter = require('events').EventEmitter;
 const connection = require('./connection');
 
-var channelName;
-var PubSub = function () {
-	var self = this;
-	var subClient = connection.connect();
-	this.pubClient = connection.connect();
-
+let channelName;
+const PubSub = function () {
+	const self = this;
 	channelName = 'db:' + nconf.get('redis:database') + ':pubsub_channel';
-	subClient.subscribe(channelName);
-
-	subClient.on('message', function (channel, message) {
-		if (channel !== channelName) {
-			return;
-		}
-
-		try {
-			var msg = JSON.parse(message);
-			self.emit(msg.event, msg.data);
-		} catch (err) {
-			winston.error(err.stack);
-		}
+
+	connection.connect().then(function (client) {
+		self.subClient = client;
+		self.subClient.subscribe(channelName);
+		self.subClient.on('message', function (channel, message) {
+			if (channel !== channelName) {
+				return;
+			}
+
+			try {
+				var msg = JSON.parse(message);
+				self.emit(msg.event, msg.data);
+			} catch (err) {
+				winston.error(err.stack);
+			}
+		});
+	});
+
+	connection.connect().then(function (client) {
+		self.pubClient = client;
 	});
 };
 

src/socket.io/index.js

@@ -16,7 +16,7 @@ const Namespaces = {};
 
 const Sockets = module.exports;
 
-Sockets.init = function (server) {
+Sockets.init = async function (server) {
 	requireModules();
 
 	const SocketIO = require('socket.io').Server;
@@ -30,7 +30,8 @@ Sockets.init = function (server) {
 		// 	io.adapter(require('./single-host-cluster'));
 		// } else if (nconf.get('redis')) {
 		if (nconf.get('redis')) {
-			io.adapter(require('../database/redis').socketAdapter());
+			const adapter = await require('../database/redis').socketAdapter();
+			io.adapter(adapter);
 		} else {
 			winston.warn('clustering detected, you should setup redis!');
 		}

src/start.js

@@ -31,7 +31,8 @@ start.start = async function () {
 		await db.initSessionStore();
 
 		const webserver = require('./webserver');
-		require('./socket.io').init(webserver.server);
+		const sockets = require('./socket.io');
+		await sockets.init(webserver.server);
 
 		if (nconf.get('runJobs')) {
 			require('./notifications').startJobs();

test/mocks/databasemock.js

@@ -152,7 +152,7 @@ before(async function () {
 
 	const webserver = require('../../src/webserver');
 	const sockets = require('../../src/socket.io');
-	sockets.init(webserver.server);
+	await sockets.init(webserver.server);
 
 	require('../../src/notifications').startJobs();
 	require('../../src/user').startJobs();