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

Gateways: Rework Account Sub/Unsub #916

Merged
merged 1 commit into from Feb 27, 2019
Merged

Gateways: Rework Account Sub/Unsub #916

merged 1 commit into from Feb 27, 2019

Conversation

kozlovic
Copy link
Member

We now send A- if an account does not exists, or if there is no
interest on a given subject and no existing subscription.
An A+ is sent if an A- was previously sent and a subscription
for this account is registered.

Signed-off-by: Ivan Kozlovic ivan@synadia.com

We now send A- if an account does not exists, or if there is no
interest on a given subject and no existing subscription.
An A+ is sent if an A- was previously sent and a subscription
for this account is registered.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic
Copy link
Member Author

@wallyqs I had to change one of the test you recently added since it was often failing on Travis (or even local with say -count 50).

Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

}
optsA.Gateway.Gateways = gateways
optsB.Gateway.Gateways = gateways
optsC.Gateway.Gateways = gateways

server.SetGatewaysSolicitDelay(100 * time.Millisecond)
defer server.ResetGatewaysSolicitDelay()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this 👍

@kozlovic kozlovic merged commit 02d6d9f into master Feb 27, 2019
@kozlovic kozlovic deleted the gw_acc_sub_unsub branch February 27, 2019 02:09
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 91.574% when pulling 18399a3 on gw_acc_sub_unsub into f68f9b1 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants