apollographql / apollographql/graphql-subscriptions

Complete event is never fired when using withFilter & graphql-redis-subscriptions

Open
#248 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
1.6k
Forks
129
PR merge metrics
No merged PRs in 30d

Description

**Server code example**

![Screenshot_13](https://user-images.githubusercontent.com/74120006/141155600-d316dade-93d7-424c-a696-e670f6535e81.png)

**Client side code example**
![Screenshot_14](https://user-images.githubusercontent.com/74120006/141155808-287ece47-3456-48b5-8fdb-c323697b0d8a.png)

**Issue**
![pingggggggggggg](https://user-images.githubusercontent.com/74120006/141156686-93cb43d9-63c4-4d98-bca2-9b9654f87f28.png)

**Dependecies**
`
"apollo-server-express": "^3.5.0",
"graphql": "^16.0.1",
"graphql-redis-subscriptions": "^2.4.0",
"graphql-subscriptions": "^2.0.0",
"graphql-ws": "^5.5.5",
"ioredis": "^4.28.0",
`

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the provided server and client setup with apollo-server-express, graphql-redis-subscriptions, graphql-subscriptions, graphql-ws, and ioredis, focusing on the withFilter path. Trace why the complete event is not fired and define done as a confirmed cause, a fix, and a regression test covering the filtered Redis subscription.

Written by the indexing model from the issue text.

Assessment

Tech stack
graphql, redis, typescript
Domain
api, backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.