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
Merge mqttv311 protocol layer with mqttv5 protocol layer #902
Conversation
Signed-off-by: wanghaemq <wangwei@emqx.io>
Signed-off-by: wanghaemq <wangwei@emqx.io>
Signed-off-by: wanghaemq <wangwei@emqx.io>
Signed-off-by: wanghaemq <wangwei@emqx.io>
Signed-off-by: wanghaemq <wangwei@emqx.io>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #902 +/- ##
==========================================
+ Coverage 65.06% 65.27% +0.21%
==========================================
Files 130 129 -1
Lines 37891 37395 -496
==========================================
- Hits 24655 24411 -244
+ Misses 13236 12984 -252 ☔ View full report in Codecov by Sentry. |
* ADD [mqtt/proto] Add some logs. Signed-off-by: wanghaemq <wangwei@emqx.io>
nni_mtx_unlock(&s->mtx); | ||
nni_msg_free(msg); | ||
// close pipe directly, no DISCONNECT for MQTTv3.1.1 | ||
// close pipe directly |
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.
shall we send a DISCONNECT here for V5
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.
I think it's okey
No description provided.