Merge pull request #258 from appsignal/daemonise-scheduler-threads

Daemonise scheduler threads

Author: unflxw

.changesets/fix-an-issue-where-using-check-ins-blocks-the-application-from-shutting-down.md

@@ -0,0 +1,6 @@
+---
+bump: patch
+type: fix
+---
+
+Fix an issue where using check-ins blocks the application from shutting down.

src/appsignal/check_in/scheduler.py

@@ -55,7 +55,7 @@ def schedule(self, event: Event) -> None:
             logger.debug(f"Scheduling {describe([event])} to be transmitted")
 
             if self.thread is None:
-                self.thread = Thread(target=self._run)
+                self.thread = Thread(target=self._run, daemon=True)
                 self.thread.start()
 
     def stop(self) -> None:
@@ -118,7 +118,7 @@ def _start_waker(self, debounce: float) -> None:
         self._stop_waker()
 
         kill = ThreadEvent()
-        thread = Thread(target=self._run_waker, args=(debounce, kill))
+        thread = Thread(target=self._run_waker, daemon=True, args=(debounce, kill))
         thread.start()
         self.waker = kill