Skip to content

Commit fd78108

Browse files
committed
Bug FIx: User settings change needs to sync to all other connected users (in case name or icon changed)
1 parent e5c8e1a commit fd78108

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

lib/api/user.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,14 @@ class Users {
9191
user: Tools.copyHashRemoveKeys( user, { password: 1, salt: 1 } )
9292
});
9393

94+
// send stub user update to all connected users
95+
var stub_user = { username: user.username, full_name: user.full_name, icon: user.icon, roles: user.roles };
96+
self.doUserBroadcastAll( 'single', { list: 'users', item: stub_user } );
97+
98+
// and send redeacted user update to user's own sockets
99+
var redacted_user = Tools.copyHashRemoveKeys( user, { password: 1, salt: 1 } );
100+
self.doUserBroadcast( user.username, 'self_update', { user: redacted_user }) ;
101+
94102
}); // storage.put
95103
} ); // loaded session
96104
}

0 commit comments

Comments
 (0)