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

MQTT_KEEP_ALIVE_TIMER define value too big #18

Open
nicokorn opened this issue Jan 4, 2022 · 1 comment
Open

MQTT_KEEP_ALIVE_TIMER define value too big #18

nicokorn opened this issue Jan 4, 2022 · 1 comment
Assignees

Comments

@nicokorn
Copy link

nicokorn commented Jan 4, 2022

The project in which the issue applies to:
https://github.com/STMicroelectronics/x-cube-azrtos-h7/tree/main/Projects/STM32H735G-DK/Applications/NetXDuo/Nx_MQTT_Client

Issue 1:
Last will message does not get triggered with the MQTT Client Project altough I added the last will registering function before connecting to the broker.

Issue 2:
After a certain time subscribed topics are having a delay of about 8 seconds after I published something somwhere else.

Solution to both issues:
According the Microsoft NetxDuo documentation the keepalive argument for the nxd_mqtt_client_connect function must be in seconds.
https://docs.microsoft.com/en-us/azure/rtos/netx-duo/netx-duo-mqtt/chapter3#nxd_mqtt_client_connect

But in the application example the keep alive define is most probably in ticks. Assuming that 1 tick is 10 ms, the 30000 resulting in 300000 ms which are 300 secconds thus 5 minutes.

#define MQTT_KEEP_ALIVE_TIMER 30000 /* Define the MQTT keep alive timer for 5 minutes */

I changed the define value to 60 and the last will message gets triggered after 60 seconds and such 8 seconds delay on subscribed topics didn't happen again.

@ALABSTM
Copy link
Contributor

ALABSTM commented Jan 5, 2022

Hi @nicokorn,

Thank you for this clear and concise report and the fix proposal. I will let our development teams know about it. We will try to get back to you as soon as possible.

With regards,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants