Skip to content

Commit 8b33f20

Browse files
Merge pull request #7088 from Countly/permission-check
[core] Add null check for permission
2 parents bafa7f9 + 256e88c commit 8b33f20

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
## Version 25.03.XX
22
Fixes:
33
- [core] fixes for changeOwner script
4+
- [core] Add null checking for user permission when opening the dashboard
45

56
Enterprise Fixes:
67
- [retention_segments] Adding null check for breakdown filtering

frontend/express/app.js

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1196,8 +1196,16 @@ Promise.all([plugins.dbConnection(countlyConfig), plugins.dbConnection("countly_
11961196
});
11971197
}
11981198
else {
1199-
var writableAppIds = member.permission._.a;
1200-
var readableAppIds = Object.keys(member.permission.r).filter(readableApp => readableApp !== 'global');
1199+
var writableAppIds = [];
1200+
if (Array.isArray(member.permission._?.a)) {
1201+
writableAppIds = member.permission._.a;
1202+
}
1203+
1204+
var readableAppIds = [];
1205+
if (member.permission.r) {
1206+
readableAppIds = Object.keys(member.permission.r).filter(readableApp => readableApp !== 'global');
1207+
}
1208+
12011209
var preparedReadableIds = [];
12021210
var preparedWritableIds = [];
12031211

0 commit comments

Comments
 (0)