-
Notifications
You must be signed in to change notification settings - Fork 30
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
Adapter resub stats #305
Adapter resub stats #305
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## AdapterSubscribeUnsubscribe #305 +/- ##
===============================================================
- Coverage 82.18% 82.09% -0.10%
===============================================================
Files 19 19
Lines 8376 8445 +69
===============================================================
+ Hits 6884 6933 +49
- Misses 1492 1512 +20
☔ View full report in Codecov by Sentry. |
* If we're a regular subscribe, build the mqtt5 operation now. Otherwise, we have to wait until | ||
* we're on the event loop thread and it's safe to query the subscription set. | ||
*/ | ||
if (options->subscription_count > 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's possible to call aws_mqtt_client_connection_subscribe_multiple
with an empty aws_array_list, the subscription_count would be zero, the subscribe_op->subscribe_op would remain null, and the subscribe multiple might trigger a resubscribe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True, will add a synchronous failure check for that case.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.