fix(federation): reset retainJob at start of run()

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>

Author: solracsf

apps/federation/lib/BackgroundJob/GetSharedSecret.php

@@ -74,6 +74,10 @@ protected function parentStart(IJobList $jobList): void {
 
 	#[\Override]
 	protected function run($argument) {
+		// The DI container caches this instance, so a prior invocation in the
+		// same cron pass can leave $retainJob = true and cause this row to be
+		// re-queued unconditionally after start() removes it.
+		$this->retainJob = false;
 		$target = $argument['url'];
 		$created = isset($argument['created']) ? (int)$argument['created'] : $this->time->getTime();
 		$currentTime = $this->time->getTime();

apps/federation/lib/BackgroundJob/RequestSharedSecret.php

@@ -87,6 +87,10 @@ protected function parentStart(IJobList $jobList): void {
 	 */
 	#[\Override]
 	protected function run($argument) {
+		// The DI container caches this instance, so a prior invocation in the
+		// same cron pass can leave $retainJob = true and cause this row to be
+		// re-queued unconditionally after start() removes it.
+		$this->retainJob = false;
 		$target = $argument['url'];
 		$created = isset($argument['created']) ? (int)$argument['created'] : $this->time->getTime();
 		$currentTime = $this->time->getTime();