From ad69758c56ad796a88ad63beaa5a28f7825c3539 Mon Sep 17 00:00:00 2001 From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com> Date: Tue, 11 Nov 2025 20:54:25 +0100 Subject: [PATCH] refactor(CleanupRedis): remove JSON decode error handling from cleanupStuckJobs method --- app/Console/Commands/CleanupRedis.php | 9 --------- 1 file changed, 9 deletions(-) diff --git a/app/Console/Commands/CleanupRedis.php b/app/Console/Commands/CleanupRedis.php index 93035e255..3af1f2c3d 100644 --- a/app/Console/Commands/CleanupRedis.php +++ b/app/Console/Commands/CleanupRedis.php @@ -394,15 +394,6 @@ private function cleanupStuckJobs($redis, string $prefix, bool $dryRun, bool $is continue; } - // Check for JSON decode errors - if ($payloadData === null || json_last_error() !== JSON_ERROR_NONE) { - $errorMsg = json_last_error_msg(); - $truncatedPayload = is_string($payload) ? substr($payload, 0, 200) : 'non-string payload'; - $this->error("Failed to decode job payload for {$keyWithoutPrefix}: {$errorMsg}. Payload: {$truncatedPayload}"); - - continue; - } - $jobClass = data_get($payloadData, 'displayName', 'Unknown'); // Prefer reserved_at (when job started processing), fallback to created_at