Merge pull request #680 from irwansyahwii/fix/limit-query

fix: limit query param to use the correct int type instead of string
panoratech · Aug 30, 2024 · 94601f6 · 94601f6
2 parents 1b143b0 + ff9e4a6
commit 94601f6
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/packages/api/src/ats/user/user.controller.ts b/packages/api/src/ats/user/user.controller.ts
@@ -8,6 +8,8 @@ import {
   Param,
   Headers,
   UseGuards,
+  UsePipes,
+  ValidationPipe,
 } from '@nestjs/common';
 import { LoggerService } from '@@core/@core-services/logger/logger.service';
 import {
@@ -57,6 +59,12 @@ export class UserController {
   @ApiPaginatedResponse(UnifiedAtsUserOutput)
   @UseGuards(ApiKeyAuthGuard)
   @Get()
+  @UsePipes(
+    new ValidationPipe({
+      transform: true,
+      transformOptions: { enableImplicitConversion: true },
+    }),
+  )
   async getUsers(
     @Headers('x-connection-token') connection_token: string,
     @Query() query: QueryDto,