fix: ConnectionMonitor is stopped by cancellation token during discon…

[herrmann-m]

Description

Refactored ConnectionMonitorThread from a Thread to a Task to support proper async execution and cancellation. Updated related logic to use a CancellationToken, and replaced Thread.Sleep with Task.Delay. Updated tests accordingly and added a new test to verify that the connection monitor task stops correctly after disconnect.

This will stop the ConnectionMonitorThread from running forever after a disconnect.

Related Issue

#246

Type of Change

• 📚 Examples / docs / tutorials / dependencies update
• 🔧 Bug fix (non-breaking change which fixes an issue)
• 🥂 Improvement (non-breaking change which improves an existing feature)
• 🚀 New feature (non-breaking change which adds functionality)
• 💥 Breaking change (fix or feature that would cause existing functionality to change)
• 🔐 Security fix

Checklist

• I've written tests (if applicable) for all new methods and classes that I created. (rake test)
• I've added documentation as necessary so users can easily use and understand this feature/fix.

[cla-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @herrmann-m on file. In order for us to review and merge your code, please sign our Contributor License Agreement to get yourself added. You'll find the CLA and more information here: https://github.com/hivemq/hivemq-community/blob/master/CONTRIBUTING.adoc#contributor-license-agreement

[SaschaBa]

Thanks @herrmann-m , this problem annoyed me since some time. Hopefully it comes in fast. ;-) (@pglombardo)

[pglombardo]

Thanks @herrmann-m & @SaschaBa! @herrmann-m have you signed the CLA yet? cla-bot is getting elderly and sometimes doesn't update. Needs replacement soon.

[herrmann-m]

Hi @pglombardo, I already singed the CLA after I created the PR

[sauroter]

Hi @herrmann-m,

could I please ask you to sign the CLA again. We had an IT issue that yours got lost. I am really sorry about this.

All the best
Georg

[herrmann-m]

Hi @sauroter,

I signed the CLA again, let me know if you need anything else.

Best regards
Michael