fix(api): use more masterKey (#941)

leancloud · Oct 19, 2023 · a44c04f · a44c04f
1 parent f560d68
commit a44c04f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/next/api/src/controller/group.ts b/next/api/src/controller/group.ts
@@ -87,7 +87,7 @@ export class GroupController {
   }
 
   @Get(':id')
-  async findOne(@Param('id', new FindModelPipe(Group)) group: Group) {
+  async findOne(@Param('id', new FindModelPipe(Group, { useMasterKey: true })) group: Group) {
     const role = await this.findGroupRole(group, { useMasterKey: true });
     const users = await role.getUsers().query().find({ useMasterKey: true });
     const userIds = users.map((u) => u.id!);

diff --git a/next/api/src/controller/ticket.ts b/next/api/src/controller/ticket.ts
@@ -14,7 +14,7 @@ import {
   UseMiddlewares,
 } from '@/common/http';
 import { ZodValidationPipe } from '@/common/pipe';
-import { customerServiceOnly } from '@/middleware';
+import { customerServiceOnly, staffOnly } from '@/middleware';
 import { UpdateData } from '@/orm';
 import router from '@/router/ticket';
 import { Ticket } from '@/model/Ticket';
@@ -27,7 +27,7 @@ const createAssociatedTicketSchema = z.object({
 @Controller({ router, path: 'tickets' })
 export class TicketController {
   @Get(':id/associated-tickets')
-  @UseMiddlewares(customerServiceOnly)
+  @UseMiddlewares(staffOnly)
   @ResponseBody(TicketListItemResponse)
   getAssociatedTickets(@Ctx() ctx: Context) {
     const ticket = ctx.state.ticket as Ticket;

diff --git a/next/api/src/controller/user.ts b/next/api/src/controller/user.ts
@@ -110,7 +110,7 @@ export class UserController {
   @Get(':id')
   @UseMiddlewares(auth, staffOnly)
   @ResponseBody(UserResponse)
-  findOne(@Param('id', new FindModelPipe(User)) user: User) {
+  findOne(@Param('id', new FindModelPipe(User, {useMasterKey: true})) user: User) {
     return user;
   }