diff --git a/packages/server/src/middleware/currentapp.js b/packages/server/src/middleware/currentapp.js index 4684bb5bae..a39d2eba0a 100644 --- a/packages/server/src/middleware/currentapp.js +++ b/packages/server/src/middleware/currentapp.js @@ -45,7 +45,7 @@ module.exports = async (ctx, next) => { const globalUser = await getCachedSelf(ctx, requestAppId) appId = requestAppId // retrieving global user gets the right role - roleId = globalUser.roleId + roleId = globalUser.roleId || roleId } // nothing more to do diff --git a/packages/server/src/middleware/tests/currentapp.spec.js b/packages/server/src/middleware/tests/currentapp.spec.js index 7bdaa929d1..d288320aea 100644 --- a/packages/server/src/middleware/tests/currentapp.spec.js +++ b/packages/server/src/middleware/tests/currentapp.spec.js @@ -127,8 +127,8 @@ describe("Current app middleware", () => { } else { expect(cookieFn).not.toHaveBeenCalled() } - expect(config.ctx.roleId).toEqual("BASIC") - expect(config.ctx.user.role._id).toEqual("BASIC") + expect(config.ctx.roleId).toEqual("PUBLIC") + expect(config.ctx.user.role._id).toEqual("PUBLIC") expect(config.ctx.appId).toEqual("app_test") expect(config.next).toHaveBeenCalled() } @@ -163,7 +163,7 @@ describe("Current app middleware", () => { return "app_test" }, setCookie: jest.fn(), - getCookie: () => ({appId: "app_test", roleId: "BASIC"}), + getCookie: () => ({appId: "app_test", roleId: "PUBLIC"}), }, constants: { Cookies: {} }, }))