Merge pull request #448 from storyofams/fix/string-body-validation

[fix] string body validation

Author: ggurkal

lib/internals/validateObject.ts

@@ -19,7 +19,7 @@ export async function validateObject(
     return value;
   }
 
-  if (value == null || (typeof value === 'string' && !value.trim().length)) {
+  if (value == null || typeof value !== 'object') {
     value = {};
   }
 

lib/pipes/validators/validation.pipe.spec.ts

@@ -65,4 +65,19 @@ describe('ValidationPipe', () => {
 
     expect(ValidationPipe()('', { metaType: DTO })).rejects.toThrowError(BadRequestException);
   });
+
+  it('Should throw for a stringified JSON body.', () => {
+    class DTO {
+      @IsNotEmpty()
+      @IsEmail()
+      public email!: string;
+
+      @IsNotEmpty()
+      public name!: string;
+    }
+
+    expect(
+      ValidationPipe()('{"email":"[email protected]","name":"Hello world"}', { metaType: DTO })
+    ).rejects.toThrowError(BadRequestException);
+  });
 });