Merge pull request #85 from Samagra-Development/hotfix/validation-bug

Fixed validations issue
Samagra-Development · Sep 1, 2023 · ac66239 · ac66239
2 parents 2d8528e + c21c4e1
commit ac66239
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/api/api.controller.ts b/src/api/api.controller.ts
@@ -7,7 +7,7 @@ import {
  Param,
  Patch,
  Post,
- Query, UnprocessableEntityException, UseInterceptors,
+ Query, UnprocessableEntityException, UseInterceptors, UsePipes, ValidationPipe,
 } from '@nestjs/common';
 import {
  SignupResponse,
@@ -50,6 +50,7 @@ export class ApiController {
  }
 
  @Get('sendOTP')
+ @UsePipes(new ValidationPipe({ transform: true }))
  async sendOTP(
  @Query() params: SendOtpDto,
  @Headers('x-application-id') applicationId?,
@@ -78,12 +79,14 @@ export class ApiController {
  }
 
  @Get('verifyOTP')
+ @UsePipes(new ValidationPipe({ transform: true }))
  async verifyOTP(@Query() params: VerifyOtpDto): Promise<any> {
  const status: SMSResponse = await this.otpService.verifyOTP(params);
  return { status };
  }
 
  @Post('login')
+ @UsePipes(new ValidationPipe({ transform: true }))
  async login(
  @Body() user: LoginDto,
  @Headers('authorization') authHeader,
@@ -314,6 +317,7 @@ export class ApiController {
  }
 
  @Patch('/changePassword/update')
+ @UsePipes(new ValidationPipe({ transform: true }))
  async changePassword(
  @Headers('authorization') authHeader,
  @Headers('x-application-id') applicationId,
@@ -327,6 +331,7 @@ export class ApiController {
  }
 
  @Post('login/otp')
+ @UsePipes(new ValidationPipe({ transform: true }))
  async loginWithOtp(
  @Body() user: LoginDto,
  @Headers('authorization') authHeader,

diff --git a/src/api/api.module.ts b/src/api/api.module.ts
@@ -1,5 +1,5 @@
 import { HttpModule } from '@nestjs/axios';
-import { Module, ValidationPipe } from '@nestjs/common';
+import { Module } from '@nestjs/common';
 import { ConfigModule, ConfigService } from '@nestjs/config';
 import { ApiController } from './api.controller';
 import { ApiService } from './api.service';
@@ -11,7 +11,6 @@ import { GupshupService } from './sms/gupshup/gupshup.service';
 import { SmsService } from './sms/sms.service';
 import got from 'got/dist/source';
 import { CdacService } from './sms/cdac/cdac.service';
-import { APP_PIPE } from '@nestjs/core';
 
 const otpServiceFactory = {
  provide: OtpService,
@@ -54,10 +53,6 @@ const otpServiceFactory = {
  otpServiceFactory,
  QueryGeneratorService,
  ConfigResolverService,
- {
- provide: APP_PIPE,
- useValue: new ValidationPipe({ transform: true }),
- },
  ],
 })
 export class ApiModule {