Skip to content
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

fix frozen subscriptions #267

Merged
merged 3 commits into from Sep 11, 2019
Merged

fix frozen subscriptions #267

merged 3 commits into from Sep 11, 2019

Conversation

@gchaincl
Copy link
Contributor

gchaincl commented Sep 11, 2019

Fix frozen subscriptions when two or more clients subscribe to events.
This happened because the current RPC client does not support more than one subscription.
The RPC Service now receives a client factory, so that it creates a new client on each subscription.

@gchaincl gchaincl requested a review from hrharder Sep 11, 2019
@gchaincl gchaincl force-pushed the bug/go-kosu/rpc-frozen-subscriptions branch from efc4452 to b6b0b78 Sep 11, 2019
gchaincl and others added 2 commits Sep 11, 2019
@gchaincl gchaincl merged commit f652fbb into master Sep 11, 2019
1 check passed
1 check passed
continuous-integration/drone/pr Build is passing
Details
@gchaincl gchaincl deleted the bug/go-kosu/rpc-frozen-subscriptions branch Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.