-
-
Notifications
You must be signed in to change notification settings - Fork 231
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
Introduce connecting and closed metadata #409
Conversation
@mcollina the PR is completed, please take you time to have a review. :) |
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.
We should document those properties
Sure. Those properties are read-only. I will use Symbol to archive that later |
@mcollina I will create another PR for using Symbols. This PR is completed. :) |
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.
lgtm
connecting
: show we are in CONNECT phase. Please noted thatconnackSent
does not mean resetconnecting
to false and setconnected
to true. We still need to procceed the last step, pushing (QoS>0 and !clean) / retain messages to clientsconnected
should be flagged true whenclientReady
/connected
is emittedclosed
: show client closed state