From 5c0538e52ca15a28d6cba3a34c3618fe3c39724c Mon Sep 17 00:00:00 2001
From: Desmond Cox <desmond@desmondgc.com>
Date: Fri, 28 Feb 2025 18:50:00 +0100
Subject: [PATCH] fix(server): stringify error log parameter to ensure correct
 overload (#16422)

* fix(server): stringify error log parameter to ensure correct overload

The intended error(message, stack, context) overload is only selected if context is a string.

* formatter
---
 server/src/services/job.service.ts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/server/src/services/job.service.ts b/server/src/services/job.service.ts
index 95ff1ad303..22408c33de 100644
--- a/server/src/services/job.service.ts
+++ b/server/src/services/job.service.ts
@@ -195,7 +195,11 @@ export class JobService extends BaseService {
         await this.onDone(job);
       }
     } catch (error: Error | any) {
-      this.logger.error(`Unable to run job handler (${queueName}/${job.name}): ${error}`, error?.stack, job.data);
+      this.logger.error(
+        `Unable to run job handler (${queueName}/${job.name}): ${error}`,
+        error?.stack,
+        JSON.stringify(job.data),
+      );
     } finally {
       this.telemetryRepository.jobs.addToGauge(queueMetric, -1);
     }