search

rebus-org / Rebus.SqlServer

:bus: Microsoft SQL Server transport and persistence for Rebus
https://mookid.dk/category/rebus
Other
44 stars 45 forks source link

Clear the timer incase Dispose() is called multiple times #31

Closed MrMDavidson closed 6 years ago

MrMDavidson commented 6 years ago

We've seen an issue a few times where the timer for the AutomaticLeaseRenewer was throwing because the timer had already been Dispose()ed when attempting to clear the timer (as part of Dispose()). For safety set the timer to NULL in Dispose().

Rebus is MIT-licensed. The code submitted in this pull request needs to carry the MIT license too. By leaving this text in, I hereby acknowledge that the code submitted in the pull request has the MIT license and can be merged with the Rebus codebase.

mookid8000 commented 6 years ago

cool 👍 it'll be out in a few minutes

mookid8000 commented 6 years ago

it's out in 5.0.0-b6 now 😄