Skip to content

Commit 19b8f0f

Browse files
committed
Fix usage of incorrect header to set count retries
1 parent 7767f30 commit 19b8f0f

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

src/CloudTasksJob.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,4 @@ public function delete(): void
9595

9696
$this->cloudTasksQueue->delete($this);
9797
}
98-
99-
public function fire(): void
100-
{
101-
$this->attempts++;
102-
103-
parent::fire();
104-
}
10598
}

src/TaskHandler.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ private function handleTask(array $task): void
9595

9696
$this->loadQueueRetryConfig($job);
9797

98-
$job->setAttempts((int) request()->header('X-CloudTasks-TaskExecutionCount'));
98+
$job->setAttempts((int) request()->header('X-CloudTasks-TaskRetryCount'));
9999
$job->setMaxTries($this->retryConfig->getMaxAttempts());
100100

101101
// If the job is being attempted again we also check if a
@@ -118,6 +118,8 @@ private function handleTask(array $task): void
118118
$job->setRetryUntil(CloudTasksApi::getRetryUntilTimestamp($fullTaskName));
119119
}
120120

121+
$job->setAttempts($job->attempts() + 1);
122+
121123
app('queue.worker')->process($this->config['connection'], $job, new WorkerOptions());
122124
}
123125

tests/TestCase.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -143,16 +143,16 @@ public function run(): void
143143
app(TaskHandler::class)->handle($this->payload);
144144
});
145145

146-
$taskExecutionCount = request()->header('X-CloudTasks-TaskExecutionCount', 0);
147-
request()->headers->set('X-CloudTasks-TaskExecutionCount', $taskExecutionCount + 1);
146+
$taskRetryCount = request()->header('X-CloudTasks-TaskRetryCount', 0);
147+
request()->headers->set('X-CloudTasks-TaskRetryCount', $taskRetryCount + 1);
148148
}
149149

150150
public function runWithoutExceptionHandler(): void
151151
{
152152
app(TaskHandler::class)->handle($this->payload);
153153

154-
$taskExecutionCount = request()->header('X-CloudTasks-TaskExecutionCount', 0);
155-
request()->headers->set('X-CloudTasks-TaskExecutionCount', $taskExecutionCount + 1);
154+
$taskRetryCount = request()->header('X-CloudTasks-TaskRetryCount', 0);
155+
request()->headers->set('X-CloudTasks-TaskRetryCount', $taskRetryCount + 1);
156156
}
157157
};
158158
}

0 commit comments

Comments
 (0)