astarte-platform / astarte-device-sdk-rust

Apache License 2.0
6 stars 12 forks source link

fix(mqtt): send purge device properties on conn #390

Closed joshuachp closed 4 weeks ago

joshuachp commented 1 month ago

Send a purge properties to Astarte on first connection to make sure the connection and properties are in good state.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 84.21053% with 6 lines in your changes missing coverage. Please review.

Project coverage is 67.0%. Comparing base (bf35d4f) to head (75c34f0). Report is 5 commits behind head on release-0.9.

Files with missing lines Patch % Lines
src/properties.rs 88.8% 3 Missing :warning:
src/transport/mqtt/connection.rs 72.7% 3 Missing :warning:
Additional details and impacted files | [Files with missing lines](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform) | Coverage Δ | | |---|---|---| | [astarte-device-sdk-derive/src/case.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=astarte-device-sdk-derive%2Fsrc%2Fcase.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-YXN0YXJ0ZS1kZXZpY2Utc2RrLWRlcml2ZS9zcmMvY2FzZS5ycw==) | `0.0% <ø> (ø)` | | | [src/builder.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Fbuilder.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2J1aWxkZXIucnM=) | `77.9% <ø> (ø)` | | | [src/interface/mapping/iter.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterface%2Fmapping%2Fiter.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9tYXBwaW5nL2l0ZXIucnM=) | `86.6% <ø> (ø)` | | | [src/interface/mapping/path.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterface%2Fmapping%2Fpath.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9tYXBwaW5nL3BhdGgucnM=) | `61.8% <ø> (ø)` | | | [src/interface/mapping/vec.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterface%2Fmapping%2Fvec.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9tYXBwaW5nL3ZlYy5ycw==) | `97.3% <ø> (ø)` | | | [src/interface/mod.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterface%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9tb2QucnM=) | `78.6% <ø> (ø)` | | | [src/interface/reference.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterface%2Freference.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZS9yZWZlcmVuY2UucnM=) | `73.6% <ø> (ø)` | | | [src/interfaces.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Finterfaces.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL2ludGVyZmFjZXMucnM=) | `78.8% <ø> (ø)` | | | [src/retention/sqlite/mod.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Fretention%2Fsqlite%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3JldGVudGlvbi9zcWxpdGUvbW9kLnJz) | `79.8% <ø> (ø)` | | | [src/transport/mqtt/mod.rs](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree&filepath=src%2Ftransport%2Fmqtt%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform#diff-c3JjL3RyYW5zcG9ydC9tcXR0L21vZC5ycw==) | `50.3% <ø> (ø)` | | | ... and [3 more](https://app.codecov.io/gh/astarte-platform/astarte-device-sdk-rust/pull/390?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astarte-platform) | |
joshuachp commented 4 weeks ago

Added the changelog