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

Implement Managed Client MQTTnet extension #70

Merged
merged 6 commits into from
Apr 26, 2024

Conversation

osmanhadzic
Copy link
Collaborator

@osmanhadzic osmanhadzic commented Apr 23, 2024

Implement extension Managed Client for MQTTNet library.
Managed Client extension implement package id, strategy for fallout messages sent to MQTT broker.

Added column guid in AstarteFailedMessages table, column is type Guid and represent messageId
Refactor strategy for fallback messages.
Remove method HandlePropertiesFailedPublish and implement interface IManagedMqttClientStorage.
Refactor method for Connect, Disconnect and DoSendMqttMessage to broker with methods from Managed Client.
Refactor methods to async methods:

  • Disconnect
  • StreamData for individual and aggregate messages

Adapt Unit and E2E tests.
Adapt SDK example.

Add new column Guid. Column Guid is used for
storing message IDs. Message ID is the unique
identification for every successful send message
to the MQTT broker.

Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
@osmanhadzic osmanhadzic force-pushed the astarte-packet-id branch 2 times, most recently from 3bd01b0 to 03fae2e Compare April 23, 2024 14:07
@osmanhadzic osmanhadzic force-pushed the astarte-packet-id branch 3 times, most recently from f732c07 to bcb1c52 Compare April 24, 2024 08:15
@harlem88 harlem88 requested a review from nedimtokic April 24, 2024 08:36
@osmanhadzic osmanhadzic force-pushed the astarte-packet-id branch 4 times, most recently from 337063c to 3b3f8dc Compare April 24, 2024 12:35
@osmanhadzic osmanhadzic marked this pull request as ready for review April 24, 2024 14:35
@osmanhadzic osmanhadzic requested a review from nedimtokic April 24, 2024 14:35
@harlem88
Copy link
Collaborator

In some commits message you write fallout strategy, Maybe did you refer to fallback strategy?

Refactor `Connect` method in
`AstarteMqttTransport`.
Refactor fallback strategy to implement
ManagedClient storage.

Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
Refactor method `Disconnect` in `AstarteDevice`
to async `Disconnect`

Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
Refactor method for `StreamData` for individual
and aggregate data to `Astarte`

Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
Update methods calls `StreamData` for individual
and aggregate object data

Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
Signed-off-by: Osman Hadzic <osman.hadzic@secomind.com>
@harlem88 harlem88 merged commit bf3ba84 into astarte-platform:master Apr 26, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants