Why required @nest/graphql for cookie setting ?

[mindonline]

In branch Nest7 GraphQL package is required by default even project don't use GraphQL.

[bkostrowiecki]

Yeah, I would like to ask the same question

[johnbiundo]

It's because it now uses the context.getType() function to determine the execution context, which could be Graphql. I'd like to remove this dependency, but don't have time right at this moment to do so. PR's welcome if someone else would like to tackle this.

[bkostrowiecki]

Hmmm, maybe I'll try, but I can't promise anything. Thanks for response.

EDIT: BTW. This version was published from the different branch than master, right? It's from nest7?

[jmcdo29]

I actually ran into something similar building up nestjs-throttler. I found a way around it though, using context.getArgsByIndex(2) to mimic GqlExecutionContext.create(context).getContext(). It does mean that Fastify is incompatible, but I don't think that affects this package.