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

[Bug Report] QOS not set for incoming messages from $iothub/twin/PATCH/properties/desired/ topic #1746

Closed
dzakub opened this issue Oct 11, 2023 · 3 comments
Assignees
Labels
bug fix checked in Fix checked into main or preview, but not yet released. IoTSDK

Comments

@dzakub
Copy link
Contributor

dzakub commented Oct 11, 2023

Context

  • OS and version used: - 6.4.12-arch1-1 GNU/Linux
  • Java runtime used: OpenJDK Runtime Environment 18.9 (build 11.0.12+7)
  • SDK version used: 2.3.0

Description of the issue

Messages received from topic: $iothub/twin/PATCH/properties/desired/ do not set QOS properly. This causes ACK message to not to be sent to a broker.
Issue was discovered while writing integration tests with fake IOT hub.

Console log of the issue

2023-10-10 17:04:16,262 TRACE - com.hivemq.mqtt.handler.publish.PublishFlowHandler - Client 'iot-hub-fake' completed a PUBLISH flow with QoS 1 or 2 for packet identifier '3'
2023-10-10 17:04:17,026 TRACE - com.microsoft.azure.sdk.iot.device.transport.mqtt.Mqtt - Mqtt message arrived on topic $iothub/twin/PATCH/properties/desired/?$version=2 with mqtt message id 151
2023-10-10 17:04:17,026 INFO  - IotHubFake - IotHubFake: Delivery complete, message id: 3, topics: [$iothub/twin/PATCH/properties/desired/?$version=2], complete: true
2023-10-10 17:06:33,465 TRACE - com.hivemq.mqtt.message.pool.SequentialMessageIDPoolImpl - Tried to return message id 51 although it was already returned. This is could mean a DUP was acked
2023-10-10 17:06:33,465 TRACE - com.microsoft.azure.sdk.iot.device.transport.mqtt.MqttIotHubConnection - Received MQTT device twin message ( Message details: Correlation Id [9410b255-3b4d-4212-bc39-a65962028587] Message Id [0d26490a-371a-474d-83ab-918aba12af34] Device Operation Type [DEVICE_OPERATION_TWIN_SUBSCRIBE_DESIRED_PROPERTIES_RESPONSE] )
2023-10-10 17:06:33,466 TRACE - com.microsoft.azure.sdk.iot.device.transport.mqtt.MqttIotHubConnection - MQTT received message with QoS 0 so it has not been added to the messages to acknowledge list ( Message details: Correlation Id [9410b255-3b4d-4212-bc39-a65962028587] Message Id [0d26490a-371a-474d-83ab-918aba12af34] Device Operation Type [DEVICE_OPERATION_TWIN_SUBSCRIBE_DESIRED_PROPERTIES_RESPONSE] )
@dzakub dzakub added the bug label Oct 11, 2023
@dzakub dzakub changed the title [Bug Report] [Bug Report] QOS not set for incomming messages from $iothub/twin/PATCH/properties/desired/ topic Oct 11, 2023
@dzakub dzakub changed the title [Bug Report] QOS not set for incomming messages from $iothub/twin/PATCH/properties/desired/ topic [Bug Report] QOS not set for incoming messages from $iothub/twin/PATCH/properties/desired/ topic Oct 11, 2023
@dzakub
Copy link
Contributor Author

dzakub commented Oct 11, 2023

I have fix completed for this issue. But I cannot push it. Do I need some permissions?

dzakub added a commit to dzakub/azure-iot-sdk-java that referenced this issue Oct 12, 2023
@timtay-microsoft timtay-microsoft self-assigned this Oct 16, 2023
@timtay-microsoft
Copy link
Member

We can discuss this issue a bit in your PR #1747

@timtay-microsoft timtay-microsoft added the fix checked in Fix checked into main or preview, but not yet released. label Oct 17, 2023
@timtay-microsoft
Copy link
Member

Version 2.3.1 has been released with this fix, so I'll close this thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix checked in Fix checked into main or preview, but not yet released. IoTSDK
Projects
None yet
Development

No branches or pull requests

2 participants