Fix: mark MaxAliasesOptions's allowList property as optional (#685)

* Fix: mark MaxAliasesOptions allowList property as optional; a default empty array is provided via maxAliasesDefaultOptions * chore(yarn changeset): ran yarn changeset
Escape-Technologies · Jul 2, 2024 · ca9f663 · ca9f663
1 parent 05fa628
commit ca9f663
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/ninety-actors-occur.md b/.changeset/ninety-actors-occur.md
@@ -0,0 +1,5 @@
+---
+'@escape.tech/graphql-armor-max-aliases': minor
+---
+
+fix: improve MaxAliasesOptions typing, mark allowList property as optional [#685](https://github.com/Escape-Technologies/graphql-armor/pull/685)
diff --git a/packages/plugins/max-aliases/src/index.ts b/packages/plugins/max-aliases/src/index.ts
@@ -11,7 +11,7 @@ import {
   ValidationContext,
 } from 'graphql';
 
-type MaxAliasesOptions = { n?: number; allowList: string[] } & GraphQLArmorCallbackConfiguration;
+type MaxAliasesOptions = { n?: number; allowList?: string[] } & GraphQLArmorCallbackConfiguration;
 const maxAliasesDefaultOptions: Required<MaxAliasesOptions> = {
   n: 15,
   onAccept: [],