Make sure 'ctx.node' is defined for the V4 adaptor

graphile · Mar 9, 2023 · 516cf0c · 516cf0c
1 parent c2bb20b
commit 516cf0c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/@app/lib/src/GraphileApolloLink.ts b/@app/lib/src/GraphileApolloLink.ts
@@ -6,10 +6,10 @@ import {
   Operation,
 } from "@apollo/client";
 import { Request, Response } from "express";
-import { getOperationAST } from "graphql";
 import { execute, hookArgs, isAsyncIterable } from "grafast";
-import type { PostGraphileInstance } from "postgraphile";
 import type {} from "grafserv/express/v4";
+import { getOperationAST } from "graphql";
+import type { PostGraphileInstance } from "postgraphile";
 
 export interface GraphileApolloLinkInterface {
   /** The request object. */
@@ -62,6 +62,10 @@ export class GraphileApolloLink extends ApolloLink {
           await hookArgs(
             args,
             {
+              node: {
+                req,
+                res,
+              },
               expressv4: {
                 req,
                 res,

diff --git a/@app/server/__tests__/helpers.ts b/@app/server/__tests__/helpers.ts
@@ -174,7 +174,11 @@ export const runGraphQLQuery = async function runGraphQLQuery(
     },
     variableValues: variables,
   };
-  await hookArgs(args, { node: { req, res } }, resolvedPreset);
+  await hookArgs(
+    args,
+    { node: { req, res }, expressv4: { req, res } },
+    resolvedPreset
+  );
 
   // Because we're connected as the database owner, we should manually switch to
   // the authenticator role