From ca9f66399e519fde6ab75f77c427ddafd27be5b3 Mon Sep 17 00:00:00 2001 From: Stephen Levi O'Neil Date: Tue, 2 Jul 2024 15:06:40 -0400 Subject: [PATCH] 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 --- .changeset/ninety-actors-occur.md | 5 +++++ packages/plugins/max-aliases/src/index.ts | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) create mode 100644 .changeset/ninety-actors-occur.md diff --git a/.changeset/ninety-actors-occur.md b/.changeset/ninety-actors-occur.md new file mode 100644 index 00000000..3d13acf8 --- /dev/null +++ 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 index e83b43b6..eaf6a5d6 100644 --- 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 = { n: 15, onAccept: [],