supernets/hardlounge
supernets/hardlounge
4
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Ensure passwords are reloaded when updated via CLI (fixes #598)

Browse Source
This commit is contained in:
Rob Jackson 2017-09-30 23:19:53 +01:00 committed by Jérémie Astori
parent 36e0ce46b4
commit 15f7f2e371
No known key found for this signature in database
GPG Key ID: B9A4F245CD67BDE8
1 changed files with 18 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/clientManager.js
View File

@ -51,8 +51,8 @@ ClientManager.prototype.autoloadUsers = function() {
log.info(noUsersWarning); log.info(noUsersWarning);
} }
// New users created since last time users were loaded // Reload all users. Existing users will only have their passwords reloaded.
_.difference(updatedUsers, loaded).forEach((name) => this.loadUser(name)); updatedUsers.forEach((name) => this.loadUser(name));
// Existing users removed since last time users were loaded // Existing users removed since last time users were loaded
_.difference(loaded, updatedUsers).forEach((name) => { _.difference(loaded, updatedUsers).forEach((name) => {
@ -67,22 +67,30 @@ ClientManager.prototype.autoloadUsers = function() {
}; };
ClientManager.prototype.loadUser = function(name) { ClientManager.prototype.loadUser = function(name) {
const user = readUserConfig(name); const userConfig = readUserConfig(name);
if (!user) { if (!userConfig) {
return; return;
} }
let client = this.findClient(name); let client = this.findClient(name);
if (client) { if (client) {
log.warn(`Tried to load user ${colors.bold(name)}, which is already loaded.`); if (userConfig.password !== client.config.password) {
return client; /**
* If we happen to reload an existing client, make super duper sure we
* have their latest password. We're not replacing the entire config
* object, because that could have undesired consequences.
*
* @see https://github.com/thelounge/lounge/issues/598
*/
client.config.password = userConfig.password;
log.info(`Password for user ${colors.bold(name)} was reset.`);
} }
} else {
client = new Client(this, name, user); client = new Client(this, name, userConfig);
this.clients.push(client); this.clients.push(client);
}
return client; return client;
}; };