-
Notifications
You must be signed in to change notification settings - Fork 261
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
Bugfix for qos change in extension affecting the publisher #302
Bugfix for qos change in extension affecting the publisher #302
Conversation
…ffecting publisher
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @Remit on file. In order for us to review and merge your code, please sign our Contributor License Agreement to get yourself added. You'll find the CLA and more information here: https://github.com/hivemq/hivemq-community/blob/master/CONTRIBUTING.adoc#contributor-license-agreement |
@cla-bot check |
The cla-bot has been summoned, and re-checked this pull request! |
Motivation
Resolves a bug when changing QoS in the inbound interceptor also affected publisher so that it got unexpected responses causing the client to disconnect.
Changes
Splitting QoS on the modifiable publish into original QoS used for publishers and modified QoS used for subscribers (onward QoS).