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
Request for persistent sessions ignored. #1272
Comments
The
|
Thanks much for the reply. Unfortunately, I can only wish that were the problem. I had to give it another try though as I have spent a bunch of hours trying to understand where the problem lies that who knows, I could have missed this case. The results are below. Client with debug
mosquitto.log
|
I'm fairly certain that this has been fixed with the same commit as #1273. If you feel like testing it, you could get the |
@ralight I'm facing the same issue too with version 1.6.2. Clean session set to false is ignored by the broker. I'm available for testing but I need you help |
@addendumE I'm just preparing the 1.6.3 release, so it might be best to test that instead. |
Hi Roger,
just downloaded 1.6.3 release but the issue is still present.
Here is my node.js test script that should report "session restored" at
second run. Using the online hivemq broker the script works fine but using
localhost (mosquitto 1.6.3) it always return "subscribed"
Maurizio
const mqtt = require ('mqtt');
var host="mqtt:https://localhost";
//var host="mqtt:https://broker.hivemq.com";
var client = mqtt.connect(host, {clientId:'SQLwriter',clean:false,});
client.on('connect', function (connAck) {
if (connAck.sessionPresent==false){
client.subscribe({'/test12345':{qos:1}}, function (err) {
console.log("subscribed");
})
}
else
{
console.log("session restored!!");
}
})
Il giorno mar 18 giu 2019 alle ore 15:05 Roger Light <
[email protected]> ha scritto:
… @addendumE <https://github.com/addendumE> I'm just preparing the 1.6.3
release, so it might be best to test that instead.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1272?email_source=notifications&email_token=ACDI5KMCM6K3X246KNON3SLP3DMQLA5CNFSM4HL6LZM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX6RSFY#issuecomment-503126295>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACDI5KL5DY3YMSEFGK5BOZDP3DMQLANCNFSM4HL6LZMQ>
.
|
I've tested this and am confident the fix I've pushed to the |
Well, dang. The 1.6.3 release just became available in the repo (repo.mosquitto.com) and I was just able to install it and test it and find that it is still not working. I come here to report it to find that it has already been found and fixed. YAY! Since I am running on a PI and don't have a development environment to compile for the PI, it seems I will have to wait for the next release. Unless, of course, there is a source for the RasPi binary of which I am not aware. |
Hi Roger,
I can confirm that the issue is solved in the fixes branch.
Maurizio
Il giorno mar 25 giu 2019 alle ore 18:24 Roger Light <
[email protected]> ha scritto:
… I've tested this and am confident the fix I've pushed to the fixes branch
solves this problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1272?email_source=notifications&email_token=ACDI5KMJM6TUOIMMJOBSZMLP4JBCXA5CNFSM4HL6LZM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYQZX3A#issuecomment-505519084>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACDI5KPTVD3CTDKZH6UEM6LP4JBCXANCNFSM4HL6LZMQ>
.
|
Closes eclipse#1272. Thanks to BowenMarmot and addendumE.
Closes #1272. Thanks to BowenMarmot and addendumE.
I upgraded to 1.6.2 on my raspberry pi. It seems that all session retention is being ignored by the broker.
I connect to the broker with qos and disable-clean-session set.
The log file shows that the CONNACK is returned with the "Session Present Flag" set to 0 which means the server did not set a persistent session.
This is supported in the tcpdump. The first bold 00 shows a "clean session" is not requested. The second bold 00 is saying that a persistent session is not supplied. The bolding is hard to see but it is there. I wish github did colours.
The text was updated successfully, but these errors were encountered: