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

Keep Alive is not treated according to MQTT standard #233

Closed
psavyhin opened this issue Nov 21, 2022 · 2 comments
Closed

Keep Alive is not treated according to MQTT standard #233

psavyhin opened this issue Nov 21, 2022 · 2 comments

Comments

@psavyhin
Copy link

It was found that coreMQTT application that intensively sends messages to a cloud can't pass Device Advisor "Keep Alive" test.
Details are described in an attached doc.
RD-MQTTKeepAlivetestissue-211122-1343.pdf

@ademyankov
Copy link

@psavyhin I noticed some weird behavior too... I think the issue here is that the FreeRTOS repo uses very old coreMQTT version 1.1.0. When I used the latest coreMQTT v.2.1.1 I don't see any issues anymore.

FreeRTOS manifest needs to be updated for the newest version. There are some minor changes in the API so the examples will probably require change as well.

@AniruddhaKanhere
Copy link
Member

@psavyhin, this issue was resolved in PR: #235

@ademyankov the FreeRTOS manifest is already updated to be beyond v2.1.1 commit. /~https://github.com/FreeRTOS/FreeRTOS/blob/main/manifest.yml#L41-L46. Did I miss something?

I am closing this issue. Please feel free to re-open this issue or create a new one if you think that there has been a mistake on my part.

Thank you.

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

3 participants