* Don't crash attempting to get desc from menu stack

* Remove ring buffer monitor at WFC exit
This commit is contained in:
Bryan Ashby 2020-11-27 23:01:05 -07:00
parent b075cbae65
commit a64bbb1635
No known key found for this signature in database
GPG Key ID: B49EB437951D2542
2 changed files with 13 additions and 1 deletions

View File

@ -36,11 +36,19 @@ function getActiveConnectionList(authUsersOnly) {
const now = moment();
return _.map(getActiveConnections(authUsersOnly), ac => {
let action;
try {
// attempting to fetch a bad menu stack item can blow up/assert
action = _.get(ac, 'currentMenuModule.menuConfig.desc', 'Unknown');
} catch(e) {
action = 'Unknown';
}
const entry = {
node : ac.node,
authenticated : ac.user.isAuthenticated(),
userId : ac.user.userId,
action : _.get(ac, 'currentMenuModule.menuConfig.desc', 'Unknown'),
action : action,
serverName : ac.session.serverName,
isSecure : ac.session.isSecure,
};

View File

@ -103,6 +103,10 @@ exports.getModule = class WaitingForCallerModule extends MenuModule {
}
leave() {
_.remove(Log.log.streams, stream => {
return stream.name === 'wfc-ringbuffer';
});
this._stopRefreshing();
super.leave();