SetDesiredPropertyUpdateCallback() is not working. about azure-iot-sdk-csharp HOT 6 CLOSED

Based on investigation, the code creates the first chain of inner handlers correctly and sets the callback. However, within the call of SetDesiredPropretyUpdateCallback(), the code resets part of the chain without setting the callback again - so it gets lost.
We did a work around where we checked if the inner handler is present before creating a new one. This unblocked us.

from azure-iot-sdk-csharp.

Jasmine pointed me to the sample, and the problem is that I’m missing the explicit call to OpenAsync()

            Client = DeviceClient.CreateFromConnectionString(DeviceConnectionString, TransportType.Mqtt);
            Client.OpenAsync().Wait(); // <-- this is missing in my repro.
            Client.SetDesiredPropertyUpdateCallback(OnDesiredPropertyChanged, null).Wait();

Once I added this, things started to work.

This makes sense, because when I debugged SetDesiredPropertyUpdateCallback(), it ended-up calling OpenAsync(), and during that call the callback was lost. In other words, the code path for OpenAsync() does not expect the callbacks to be set, and overwrites them.

So, if SetDesiredPropertyUpdateCallback() is expected to handle the case where OpenAsync() hasn't been called yet - we might still have a bug.

from azure-iot-sdk-csharp.

The is a valid bug and SetDesiredPropertyUpdateCallback() is expected to handle the case where OpenAsync() hasn't been called. Fix is on the way. Thanks.

from azure-iot-sdk-csharp.

commit: ccd8bc9

from azure-iot-sdk-csharp.

I'm trying to run the sample (https://github.com/Azure/azure-iot-sdk-csharp/blob/master/device/samples/DeviceClientTwinMqttSample/Program.cs), but it crashes on:

Client.SetDesiredPropertyUpdateCallback(OnDesiredPropertyChanged, null).Wait();

Connecting to hub

Error in sample: System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.Azure.Devices.Client.Transport.TransportHandlerFactory.Create(IPipelineContext context)
at Microsoft.Azure.Devices.Client.DeviceClient.<>c__DisplayClass24_0.b__4(IPipelineContext ctx)
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.EnsureInnerHandlerInitialized()
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.get_InnerHandler()
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.set_TwinUpdateHandler(TwinUpdateCallback value)
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.set_TwinUpdateHandler(TwinUpdateCallback value)
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.set_TwinUpdateHandler(TwinUpdateCallback value)
at Microsoft.Azure.Devices.Client.Transport.DefaultDelegatingHandler.set_TwinUpdateHandler(TwinUpdateCallback value)
at Microsoft.Azure.Devices.Client.DeviceClient.<>c__DisplayClass60_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Devices.Client.TaskHelpers.d__17.MoveNext()

The only modification I did to the code was to set the connection string for my device on the form:

"HostName=<iothub_host_name>;DeviceId=<device_id>;SharedAccessKey=<device_key>"

using my real connection string for the device of course.

I'm using v1.2.2 of the Microsoft.Azure.Devices.Client

from azure-iot-sdk-csharp.

Fix provided in 1.2.3. Please let us know if still having issue.

from azure-iot-sdk-csharp.