Skip to content
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

max_inflight_messages=1 broken on Mosquitto-1.6.3 #1332

Closed
xmas79 opened this issue Jul 5, 2019 · 2 comments
Closed

max_inflight_messages=1 broken on Mosquitto-1.6.3 #1332

xmas79 opened this issue Jul 5, 2019 · 2 comments

Comments

@xmas79
Copy link

xmas79 commented Jul 5, 2019

Hello,
it seems that max_inflight_messages=1 is broken on the new mosquitto-1.6.3 broker implementation. A client (tried with mosquitto_pub v1.6.3 and older releases as well) publishing messages with QoS2 to a such configured broker will just stuck waiting for a reply.

As I understand it, the problem seems related to outgoing QoS quota being decremented for the CONNACK packet, leaving the broker without any quota left for answering subsequent requests.

A simple "dialogue" between a broker and a client:

Log from Mosquitto Broker:

1562328636: New connection from 192.168.0.87 on port 1883.
1562328636: New client connected from 192.168.0.87 as mosquitto_pub (p2, c1, k60).
1562328636: No will message specified.
1562328636: Sending CONNACK to mosquitto_pub (0, 0)
1562328636: Received PUBLISH from mosquitto_pub (d0, q2, r0, m1, 'test', ... (5 bytes))
1562328696: Received PINGREQ from mosquitto_pub
1562328696: Sending PINGRESP to mosquitto_pub

Log from mosquitto_pub:

mosquitto_pub -h 192.168.0.88 -t "test" -q 2 -d -m "hello" -i "mosquitto_pub"
Client mosquitto_pub sending CONNECT
Client mosquitto_pub received CONNACK (0)
Client mosquitto_pub sending PUBLISH (d0, q2, r0, m1, 'test', ... (5 bytes))
Client mosquitto_pub sending PINGREQ
Client mosquitto_pub received PINGRESP
@ilovemilk
Copy link

I have the same problem. The issue also exists in version 1.6.2.

ralight added a commit that referenced this issue Jul 30, 2019
@ralight
Copy link
Contributor

ralight commented Jul 30, 2019

Thanks for the reports both, I believe this is now fixed in the fixes branch.

@ralight ralight added this to the 1.6.4 milestone Jul 30, 2019
@ralight ralight closed this as completed Aug 1, 2019
vankxr pushed a commit to vankxr/mosquitto that referenced this issue Aug 9, 2019
ralight added a commit that referenced this issue Sep 18, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants