We now only validate body, params, and query

iyobo · May 1, 2021 · fa664d6 · fa664d6
1 parent 942cdc1
commit fa664d6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/util/generateRoutes.ts b/src/util/generateRoutes.ts
@@ -87,8 +87,10 @@ async function _determineArgument(
     //TODO: implement custom function capability here for arg injectors
   }
 
-  // validate if this is a class. If it is a class, validate it
-  if (values && isClass(type)) {
+  // validate if this is a class and if this is a body, params, or query injection
+  const shouldValidate = values && isClass(type) && ['body','params','query'].includes(injectSource)
+
+  if (shouldValidate) {
     values = await plainToClass(type, values);
 
     const errors = await validate(values, options.validatorOptions); // TODO: wrap around this to trap runtime errors

diff --git a/src/util/tools.ts b/src/util/tools.ts
@@ -2,7 +2,7 @@ import {metadata} from '../index';
 import {ValidationDecoratorOptions} from '../decorators';
 
 export function isClass(type) {
-  return type?.prototype?.constructor;
+  return type?.prototype?.constructor && type?.prototype?.constructor?.name !== 'Object';
 }
 
 interface AddArgumentProps {