From a26dcf507536588ce62d41252da84fb436c145cb Mon Sep 17 00:00:00 2001 From: Luciole <31817566+Luciole0x@users.noreply.github.com> Date: Mon, 14 Aug 2023 13:31:55 +0200 Subject: [PATCH] defaultRoleId: explicitly check for null --- packages/controller/src/UserManager.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/controller/src/UserManager.ts b/packages/controller/src/UserManager.ts index d964aa69..bd2789c3 100644 --- a/packages/controller/src/UserManager.ts +++ b/packages/controller/src/UserManager.ts @@ -71,8 +71,11 @@ export default class UserManager { throw new Error(`User '${name}' already exists`); } + let roles = []; let defaultRoleId = this._config.get("controller.default_role_id"); - let roles = defaultRoleId ? [defaultRoleId] : []; + if (defaultRoleId !== null) { + roles.push(defaultRoleId) + } let user = new lib.User({ name, roles }, this.roles); this.users.set(name, user);