Skip to content

fix: guard mqtt cleanup on setup errors#95

Merged
gavinying merged 1 commit intomasterfrom
fix/mqtt-setup-cleanup
Dec 6, 2025
Merged

fix: guard mqtt cleanup on setup errors#95
gavinying merged 1 commit intomasterfrom
fix/mqtt-setup-cleanup

Conversation

@gavinying
Copy link
Owner

Summary

  • avoid raising secondary exceptions when MQTT setup fails by guarding close()
  • log and ignore close errors so original setup/connect failure surfaces

Test plan

  • poetry run pytest tests/test_mqtt_task.py::test_mqtt_task_setup

@gavinying gavinying merged commit daa8c0c into master Dec 6, 2025
8 checks passed
@gavinying gavinying deleted the fix/mqtt-setup-cleanup branch December 6, 2025 03:23
This was referenced Dec 6, 2025
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.

1 participant