Skip to content

fix: close redis connections on unsubscribe (upstream #2917)#123

Merged
ehaynes99 merged 3 commits into
mainfrom
pr-2917
Jun 4, 2026
Merged

fix: close redis connections on unsubscribe (upstream #2917)#123
ehaynes99 merged 3 commits into
mainfrom
pr-2917

Conversation

@ehaynes99

Copy link
Copy Markdown

Cherry-picks upstream wundergraph#2917 onto our fork.

Closes the redis pub/sub connection on unsubscribe (sub.Close()), fixing the connection leak. Also gives the unsubscribe a fixed 1s timeout context.

Upstream PR: wundergraph#2917

🤖 Generated with Claude Code

@github-actions github-actions Bot added the router label Jun 4, 2026
@ehaynes99 ehaynes99 merged commit 75e370a into main Jun 4, 2026
24 of 27 checks passed
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

Router-nonroot image scan failed

❌ Security vulnerabilities found in image:

ghcr.io/OpenPhone/cosmo/router:sha-124765b90fa811b3c5b53f544ba502e2806b5d01-nonroot

Please check the security vulnerabilities found in the PR.

If you believe this is a false positive, please add the vulnerability to the .trivyignore file and re-run the scan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants