Subscription path doesn't respect useGlobalPrefix

[Jake-RoundrockIO]

Is there an existing issue for this?

• I have searched the existing issues

Current behavior

When creating a GraphQL server with Nestjs, if you set a global prefix using app.setGlobalPrefix in main.ts and then set useGlobalPrefix: true in the GraphQLModule configuration, the global prefix is respected for Query and Mutation types but not for Subscription types.

Minimum reproduction code

https://github.com/Jake-RoundrockIO/nest-subscription-global-prefix

Steps to reproduce

1. npm install
2. npm run start:dev
3. Open http://localhost:5000/api/graphql in the browser to access the Apollo Sandbox
4. Run the following query to ensure you are able to communicate with the server.

query Recipes {
  recipes {
    id
    title
    description
    creationDate
    ingredients
  }
}

5. Run the following to test a subscription. By default, the sandbox will use ws://localhost:5000/api/graphql, which is the expected subscription endpoint.

subscription RecipeAdded {
  recipeAdded {
    id
    title
    description
    creationDate
    ingredients
  }
}

6. Upon failure to connect, modify the subscription endpoint in the sandbox to ws://localhost:5000/graphql (The endpoint minus the global /api prefix). This should properly instantiate the subscription connection, just on the wrong url path.

Expected behavior

The expected behavior is for the subscription path, whether it be the default /graphql or any other custom path, to be appended to the global prefix when the useGlobalPrefix option is set to true.

Package version

10.1.5

Graphql version

graphql: 16.6.0
apollo-server-express: 3.10.3
graphql-ws: 5.11.2

NestJS version

9.1.6

Node.js version

16.14.0

In which operating systems have you tested?

• macOS
• Windows
• Linux

Other

No response

[kamilmysliwiec]

Would you like to create a PR for this issue?