diff --git a/packages/server/src/api/controllers/permission.js b/packages/server/src/api/controllers/permission.js index e395057651..372e804958 100644 --- a/packages/server/src/api/controllers/permission.js +++ b/packages/server/src/api/controllers/permission.js @@ -71,10 +71,11 @@ async function updatePermissionOnRole( } const response = await db.bulkDocs(docUpdates) - return response.map(resp => ({ - ...resp, - _id: getExternalRoleID(resp._id), - })) + return response.map(resp => { + resp._id = getExternalRoleID(resp.id) + delete resp.id + return resp + }) } exports.fetchBuiltin = function(ctx) { diff --git a/packages/server/src/api/index.js b/packages/server/src/api/index.js index 5b9b51363b..7b063cb522 100644 --- a/packages/server/src/api/index.js +++ b/packages/server/src/api/index.js @@ -47,6 +47,7 @@ router.use(async (ctx, next) => { message: err.message, status: ctx.status, } + console.trace(err) } }) diff --git a/packages/server/src/api/routes/tests/permissions.spec.js b/packages/server/src/api/routes/tests/permissions.spec.js index d4ca9aa8a2..fc5ce23f96 100644 --- a/packages/server/src/api/routes/tests/permissions.spec.js +++ b/packages/server/src/api/routes/tests/permissions.spec.js @@ -48,7 +48,7 @@ describe("/permission", () => { describe("test", () => { it("should be able to add permission to a role for the table", async () => { expect(perms.length).toEqual(1) - expect(perms[0].id).toEqual(`${STD_ROLE_ID}`) + expect(perms[0]._id).toEqual(`${STD_ROLE_ID}`) }) it("should get the resource permissions", async () => {