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.

Sign up for GitHub

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

Jump to bottom

fix: Wait for MQTT subscriptions to take effect before returning (#516) #517

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: Wait for MQTT subscriptions to take effect before returning (#516) #517

wants to merge 2 commits into from
+7 −0

Commits on Oct 23, 2024

  1. fix: Wait for MQTT subscriptions to take effect before returning (edg… 

    …exfoundry#516)

MQTTAsync_subscribe() is async. There are some places e.g. startConfigured()
where we expect the subscription to be in effect right after this call.
So wait for completion before returning.

Fixes: edgexfoundry#516

Signed-off-by: Corey Mutter <[email protected]>
    @eaton-coreymutter
    eaton-coreymutter committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    d2bfa6c View commit details
    Browse the repository at this point in the history

Commits on Nov 26, 2024

  1. fix: Add a check for the return code of MQTTAsync_waitForCompletion 

    Signed-off-by: FelixTing <[email protected]>
    @FelixTing
    FelixTing committed Nov 26, 2024
    Configuration menu
    Copy the full SHA
    c3442cc View commit details
    Browse the repository at this point in the history