Merge PR #503

Since the refactoring 7af66b0 the Cheroot worker threads are named like `('CP Server Thread-<id>',)`. A leftover comma transformed the string to a tuple. This patch fixes this by removing the trailing comma when setting the thread name as `str`. Ref: * #503 * #516 Co-Authored-By: Carey Metcalfe <[email protected]> Co-Authored-By: Christoph Heer <[email protected]>
cherrypy · Oct 1, 2022 · a41a557 · a41a557
2 parents 2404795 + 6b7c030
commit a41a557
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cheroot/workers/threadpool.py b/cheroot/workers/threadpool.py
@@ -178,7 +178,7 @@ def start(self):
         for worker in self._threads:
             worker.name = (
                 'CP Server {worker_name!s}'.
-                format(worker_name=worker.name),
+                format(worker_name=worker.name)
             )
             worker.start()
         for worker in self._threads:
@@ -228,7 +228,7 @@ def _spawn_worker(self):
         worker = WorkerThread(self.server)
         worker.name = (
             'CP Server {worker_name!s}'.
-            format(worker_name=worker.name),
+            format(worker_name=worker.name)
         )
         worker.start()
         return worker