Fix Jobs example. (CA-262)

espressif / esp-aws-iot

AWS IoT SDK for ESP32 based chipsets

Apache License 2.0

256 stars 154 forks source link

Fix Jobs example. (CA-262) #158

Closed jasonpcarroll closed 1 year ago

jasonpcarroll commented 1 year ago

This PR fixes:

An infinite recursion that can occur when receiving a jobs message while in the middle of processing another one by offloading handling job messages from the coreMQTT event callback to the main demo task.
Fixes incorrect logic for checking errors on unsubscribe.
Fixes the demo retry logic.
Fixes printing a warning when a timestamp message is received.

This code was tested using an ESP32-S2 on ESP-IDF v5.0.

CLAassistant commented 1 year ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Jason Carroll seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}