Skip to content

Prevent starting second subscription with the same id #520

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

gklijs
Copy link
Contributor

@gklijs gklijs commented Dec 17, 2019

#141 # 📝 Description
Prevent starting second subscription with the same id, remove he complete message as it's not needed, and probably preventing flux from closing. Validated this will solve the problems, as it finally ran longer than 2 minutes.

🔗 Related Issues

#499

…lete message as it's not needed, and probably preventing flux from closing.
@smyrick smyrick dismissed their stale review December 18, 2019 20:59

This does follow the spec for subscriptions

@smyrick smyrick changed the title Prevent starting second subscription with the same id, remove he comp… Prevent starting second subscription with the same id Dec 18, 2019
@smyrick smyrick added changes: patch Changes require a patch version type: enhancement New feature or request labels Dec 18, 2019
@smyrick smyrick merged commit 9d0ace4 into ExpediaGroup:master Dec 18, 2019
@smyrick
Copy link
Contributor

smyrick commented Dec 18, 2019

Thanks for your help on the subscriptions @gklijs. If anyone else has any comments or bugs on subscriptions please comment below. We are not currently using them at ExpediaGroup so we don't have the traffic to verify but we still want this library to support subscriptions 100%

dariuszkuc pushed a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
…lete message as it's not needed, and probably preventing flux from closing. (ExpediaGroup#520)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes: patch Changes require a patch version type: enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants