use validation-pipe globally for all inputs

bhaidar · Apr 13, 2019 · af4e8e4 · af4e8e4
1 parent a0d6a89
commit af4e8e4
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 9 deletions.
diff --git a/server/src/auth/auth.controller.ts b/server/src/auth/auth.controller.ts
@@ -5,7 +5,6 @@ import {
   HttpException,
   HttpStatus,
   UsePipes,
-  ValidationPipe,
   Get,
   Req,
   UseGuards,
@@ -23,7 +22,6 @@ export class AuthController {
   constructor(private readonly authService: AuthService) {}
 
   @Post('register')
-  @UsePipes(new ValidationPipe())
   public async register(
     @Body() createUserDto: CreateUserDto,
   ): Promise<RegistrationStatus> {
@@ -39,7 +37,6 @@ export class AuthController {
   }
 
   @Post('login')
-  @UsePipes(new ValidationPipe())
   public async login(@Body() loginUserDto: LoginUserDto): Promise<LoginStatus> {
     return await this.authService.login(loginUserDto);
   }

diff --git a/server/src/main.ts b/server/src/main.ts
@@ -2,7 +2,7 @@ import 'dotenv/config';
 
 import { NestFactory } from '@nestjs/core';
 import { AppModule } from './app.module';
-import { Logger } from '@nestjs/common';
+import { Logger, ValidationPipe } from '@nestjs/common';
 import { getDbConnectionOptions, runDbMigrations } from '@shared/utils';
 
 const port = process.env.PORT;
@@ -12,6 +12,15 @@ async function bootstrap() {
     AppModule.forRoot(await getDbConnectionOptions(process.env.NODE_ENV)),
   );
 
+  /**
+   * Apply validation for all inputs globally
+   */
+  app.useGlobalPipes(
+    new ValidationPipe({
+      whitelist: true,
+    }),
+  );
+
   /**
    * Run DB migrations
    */

diff --git a/server/src/todo/task/task.controller.ts b/server/src/todo/task/task.controller.ts
@@ -5,7 +5,6 @@ import {
   Post,
   Body,
   Delete,
-  ValidationPipe,
   UsePipes,
   UseGuards,
 } from '@nestjs/common';
@@ -31,7 +30,6 @@ export class TaskController {
   }
 
   @Post('todo/:id')
-  @UsePipes(new ValidationPipe())
   @UseGuards(AuthGuard())
   async create(
     @Param('id') todo: string,

diff --git a/server/src/todo/todo.controller.ts b/server/src/todo/todo.controller.ts
@@ -6,7 +6,6 @@ import {
   Body,
   Put,
   Delete,
-  ValidationPipe,
   UsePipes,
   UseGuards,
   Req,
@@ -34,7 +33,6 @@ export class TodoController {
   }
 
   @Post()
-  @UsePipes(new ValidationPipe())
   @UseGuards(AuthGuard())
   async create(
     @Body() createTodoDto: CreateTodoDto,
@@ -46,7 +44,6 @@ export class TodoController {
   }
 
   @Put(':id')
-  @UsePipes(new ValidationPipe())
   @UseGuards(AuthGuard())
   async update(
     @Param('id') id: string,