NordicSemiconductor / IOS-nRF-Connect-Device-Manager

A mobile management library for devices supporting nRF Connect Device Manager.
https://www.nordicsemi.com/Software-and-tools/Software/nRF-Connect-SDK
Apache License 2.0
97 stars 41 forks source link

Rase condition observed for McuManager.send function's response #30

Closed ramunasjurgilas closed 2 years ago

ramunasjurgilas commented 2 years ago

I am seeding two group commands one after another:

verbose -> default -> Sending read command (Group: peruser(value: 71), ID: 0): ["limit": 5, "offset": 0]
verbose -> default -> Sending read command (Group: peruser(value: 72), ID: 0): nil

Then getting not expected response for group 72 (it comes from 71st response):

verbose -> default -> Response (Group: peruser(value: 72), ID: 0): {"rc" : 8}

An app gets wrong response for 72nd group. For me it looks response comes from the 71st group.

Could you please confirm/disconfirm, that 72nd response should be overwritten by 71st response?

dinesharjani commented 2 years ago

Hi @ramunasjurgilas is this happening on the current master branch / 1.1 version ?

ramunasjurgilas commented 2 years ago

It was happening not so often. And I can not remember on which version I got noticed it. As well problem can be in firmware it self or an app. For now I will close.