Skip to content

Fix clients not being removed from map during shutdown of application#29

Merged
kaushalaroraharman merged 1 commit into
mainfrom
bugfix/remove-mqtt-client-from-map-during-shutdown
May 14, 2025
Merged

Fix clients not being removed from map during shutdown of application#29
kaushalaroraharman merged 1 commit into
mainfrom
bugfix/remove-mqtt-client-from-map-during-shutdown

Conversation

@kaushalaroraharman
Copy link
Copy Markdown
Contributor

Resolves #28


Describe behaviour before the change

  • Exception when attempting to add null value in a ConcurrentHashMap

Describe behaviour after the change

  • Entry is removed from map, instead of adding null value during clean of mqtt client map

Pull request checklist

  • I have read the CONTRIBUTING.md
  • My code follows the code style of this project
  • Tests for the changes have been added (for bug fixes / features)
  • All new and existing tests passed.
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

  • Yes
  • No

@kaushalaroraharman kaushalaroraharman added the bug Something isn't working label May 14, 2025
@kaushalaroraharman kaushalaroraharman added this to the 1.0.1 milestone May 14, 2025
@kaushalaroraharman kaushalaroraharman self-assigned this May 14, 2025
@kaushalaroraharman kaushalaroraharman merged commit 7af0113 into main May 14, 2025
8 of 10 checks passed
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Fix null value inserted into mqtt client map

1 participant