-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[ADDED] Config options to disable MQTT QOS2 support #4705
Conversation
Changed: moved MQTT QOS2 disable config options from sess to client Added tests Signed-off-by: Lev Brouk <[email protected]>
Where are we on this? |
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.
Understood for the naming of the options. You can do the other changes (the , _
and the expect disconnect) without adding the new tests, or merge as-is if pressed by time.
@derekcollison finishing the extra tests that @kozlovic requested (I agree, they are needed), give me a few minutes |
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.
Small copy/paste issues related to NATS connection and comment, but other than that, LGTM.
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
Added MQTT config options
reject_qos2_publish
anddowngrade_qos2_subscriptions
. When used together, fully disable QOS2 support (effectively going back to v2.9 level of QOS support).