fix: fix graphql api error with apollo subscriptions (#1137)

accounts-js · May 6, 2021 · 7726eaf · 7726eaf
1 parent 217004d
commit 7726eaf
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/.changeset/silly-poems-mix.md b/.changeset/silly-poems-mix.md
@@ -0,0 +1,5 @@
+---
+'@accounts/graphql-api': patch
+---
+
+Fix `accountsGraphQL.context` breaking apollo subscriptions.
diff --git a/packages/graphql-api/src/modules/accounts/index.ts b/packages/graphql-api/src/modules/accounts/index.ts
@@ -18,6 +18,7 @@ import { CoreAccountsModule } from '../core';
 
 export interface AccountsRequest {
   req: IncomingMessage;
+  connection?: any;
 }
 
 export interface AccountsModuleConfig {

diff --git a/packages/graphql-api/src/utils/context-builder.ts b/packages/graphql-api/src/utils/context-builder.ts
@@ -3,10 +3,15 @@ import { getClientIp } from 'request-ip';
 import { AccountsRequest, AccountsModuleConfig } from '../modules';
 
 export const context = (moduleName: string) => async (
-  { req }: AccountsRequest,
+  { req, connection }: AccountsRequest,
   _: any,
   { injector }: ModuleSessionInfo
 ) => {
+  // If connection is set it means it's a websocket connection coming from apollo
+  if (connection) {
+    return connection.context;
+  }
+
   if (!req) {
     return {
       ip: '',