enhance(delegate): skip validation by default and add validateRequest…

… option to enable
ardatan · Jun 30, 2021 · aa43054 · aa43054
1 parent 672833b
commit aa43054
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/pink-glasses-move.md b/.changeset/pink-glasses-move.md
@@ -0,0 +1,5 @@
+---
+'@graphql-tools/delegate': major
+---
+
+BREAKING CHANGE: validations are skipped by default, use validateRequest: true to reenable
diff --git a/packages/delegate/src/delegateToSchema.ts b/packages/delegate/src/delegateToSchema.ts
@@ -97,7 +97,7 @@ export function delegateRequest<TContext = Record<string, any>, TArgs = any>(
 
   const processedRequest = transformer.transformRequest(options.request);
 
-  if (!options.skipValidation) {
+  if (options.validateRequest) {
     validateRequest(delegationContext, processedRequest.document);
   }
 

diff --git a/packages/delegate/src/types.ts b/packages/delegate/src/types.ts
@@ -78,7 +78,7 @@ export interface IDelegateToSchemaOptions<TContext = Record<string, any>, TArgs
   rootValue?: Record<string, any>;
   transforms?: Array<Transform<any, TContext>>;
   transformedSchema?: GraphQLSchema;
-  skipValidation?: boolean;
+  validateRequest?: boolean;
   skipTypeMerging?: boolean;
   binding?: DelegationBinding<TContext>;
 }