From 3a17a7c98fa0bd84969de478a1729f23ea3bc1c6 Mon Sep 17 00:00:00 2001
From: Jon Chambers <jon@signal.org>
Date: Thu, 4 Feb 2021 18:53:48 -0500
Subject: [PATCH] Explicitly make the dynamic config worker a daemon thread.

---
 .../textsecuregcm/storage/DynamicConfigurationManager.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/storage/DynamicConfigurationManager.java b/service/src/main/java/org/whispersystems/textsecuregcm/storage/DynamicConfigurationManager.java
index 08fc1b0a0..542ba7384 100644
--- a/service/src/main/java/org/whispersystems/textsecuregcm/storage/DynamicConfigurationManager.java
+++ b/service/src/main/java/org/whispersystems/textsecuregcm/storage/DynamicConfigurationManager.java
@@ -72,7 +72,7 @@ public class DynamicConfigurationManager {
       this.notifyAll();
     }
 
-    new Thread(() -> {
+    final Thread workerThread = new Thread(() -> {
       while (true) {
         try {
           retrieveDynamicConfiguration().ifPresent(configuration::set);
@@ -82,7 +82,10 @@ public class DynamicConfigurationManager {
 
         Util.sleep(5000);
       }
-    }).start();
+    }, "DynamicConfigurationManagerWorker");
+
+    workerThread.setDaemon(true);
+    workerThread.start();
   }
 
   private Optional<DynamicConfiguration> retrieveDynamicConfiguration() throws JsonProcessingException {