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

Fix freeze after publishing message with AWS IoT SDK v2 #173

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

elad-bar
Copy link
Collaborator

  • Update manifest requirements to use only AWS IoT Device SDK v2
  • Fix parameter of callback to the right convention
  • Fix AWS IoT MQTT publish message validation and QoS
  • Change clean_session parameter to start a new session on every attempt to connect
  • Set the AWS Client ID to the entry ID instead of randomize on every attempt to connect
  • Set message being sent while publishing empty message to empty JSON instead of empty string

@sh00t2kill sh00t2kill merged commit 5832bf6 into develop Feb 17, 2024
1 check passed
sh00t2kill pushed a commit that referenced this pull request Feb 19, 2024
* set logging of API data at DEBUG log level

* Upgrade AWS IoT Device SDK to v2

* Fix aws iot termination process

* Fix freeze after publishing message with AWS IoT SDK v2

* Fix freeze after publishing message with AWS IoT SDK v2 (#173)

* full async implrementation of publish message

* fix lint error

---------

Co-authored-by: Gil Peeters <[email protected]>
sh00t2kill added a commit that referenced this pull request Feb 19, 2024
* Fix freeze while publishing messages to AWS IoT SDK v2 (#175)

* set logging of API data at DEBUG log level

* Upgrade AWS IoT Device SDK to v2

* Fix aws iot termination process

* Fix freeze after publishing message with AWS IoT SDK v2

* Fix freeze after publishing message with AWS IoT SDK v2 (#173)

* full async implrementation of publish message

* fix lint error

---------

Co-authored-by: Gil Peeters <[email protected]>

* Update README.md

* Update README.md

---------

Co-authored-by: Elad Bar <[email protected]>
Co-authored-by: Gil Peeters <[email protected]>
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.

2 participants