Added random delay to allows delaying cron trigger startups by random amount of seconds

vtsykun · vtsykun · commit 59df6f9ff93f · 2023-03-25T11:16:01.000+01:00
diff --git a/src/Symfony/Component/Scheduler/RecurringMessage.php b/src/Symfony/Component/Scheduler/RecurringMessage.php
@@ -41,9 +41,9 @@ public static function every(string $frequency, object $message, string|\DateTim
         return new self(new DateIntervalTrigger($interval, $from, $until), $message);
     }
 
-    public static function cron(string $expression, object $message): self
+    public static function cron(string $expression, object $message, int $randomDelay = 0): self
     {
-        return new self(CronExpressionTrigger::fromSpec($expression), $message);
+        return new self(CronExpressionTrigger::fromSpec($expression, $randomDelay), $message);
     }
 
     public static function trigger(TriggerInterface $trigger, object $message): self
diff --git a/src/Symfony/Component/Scheduler/Trigger/CronExpressionTrigger.php b/src/Symfony/Component/Scheduler/Trigger/CronExpressionTrigger.php
@@ -25,20 +25,26 @@ final class CronExpressionTrigger implements TriggerInterface
 {
     public function __construct(
         private CronExpression $expression = new CronExpression('* * * * *'),
+        private readonly int $randomDelay = 0
     ) {
     }
 
-    public static function fromSpec(string $expression = '* * * * *'): self
+    public static function fromSpec(string $expression = '* * * * *', int $randomDelay = 0): self
     {
         if (!class_exists(CronExpression::class)) {
             throw new LogicException(sprintf('You cannot use "%s" as the "cron expression" package is not installed; try running "composer require dragonmantank/cron-expression".', __CLASS__));
         }
 
-        return new self(new CronExpression($expression));
+        return new self(new CronExpression($expression), $randomDelay);
     }
 
     public function getNextRunDate(\DateTimeImmutable $run): ?\DateTimeImmutable
     {
-        return \DateTimeImmutable::createFromMutable($this->expression->getNextRunDate($run));
+        $nextRun = $this->expression->getNextRunDate($run);
+        if ($this->randomDelay > 0) {
+            $nextRun->add(new \DateInterval('PT'.random_int(0, $this->randomDelay).'S'));
+        }
+
+        return \DateTimeImmutable::createFromMutable($nextRun);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -41,9 +41,9 @@ public static function every(string $frequency, object $message, string\|\DateTim`
`41`	`41`	`return new self(new DateIntervalTrigger($interval, $from, $until), $message);`
`42`	`42`	`}`
`43`	`43`
`44`		`- public static function cron(string $expression, object $message): self`
	`44`	`+ public static function cron(string $expression, object $message, int $randomDelay = 0): self`
`45`	`45`	`{`
`46`		`- return new self(CronExpressionTrigger::fromSpec($expression), $message);`
	`46`	`+ return new self(CronExpressionTrigger::fromSpec($expression, $randomDelay), $message);`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`public static function trigger(TriggerInterface $trigger, object $message): self`
Original file line number	Diff line number	Diff line change
`@@ -25,20 +25,26 @@ final class CronExpressionTrigger implements TriggerInterface`
`25`	`25`	`{`
`26`	`26`	`public function __construct(`
`27`	`27`	`private CronExpression $expression = new CronExpression('* * * * *'),`
	`28`	`+ private readonly int $randomDelay = 0`
`28`	`29`	`) {`
`29`	`30`	`}`
`30`	`31`
`31`		`- public static function fromSpec(string $expression = '* * * * *'): self`
	`32`	`+ public static function fromSpec(string $expression = '* * * * *', int $randomDelay = 0): self`
`32`	`33`	`{`
`33`	`34`	`if (!class_exists(CronExpression::class)) {`
`34`	`35`	`throw new LogicException(sprintf('You cannot use "%s" as the "cron expression" package is not installed; try running "composer require dragonmantank/cron-expression".', __CLASS__));`
`35`	`36`	`}`
`36`	`37`
`37`		`- return new self(new CronExpression($expression));`
	`38`	`+ return new self(new CronExpression($expression), $randomDelay);`
`38`	`39`	`}`
`39`	`40`
`40`	`41`	`public function getNextRunDate(\DateTimeImmutable $run): ?\DateTimeImmutable`
`41`	`42`	`{`
`42`		`- return \DateTimeImmutable::createFromMutable($this->expression->getNextRunDate($run));`
	`43`	`+ $nextRun = $this->expression->getNextRunDate($run);`
	`44`	`+ if ($this->randomDelay > 0) {`
	`45`	`+ $nextRun->add(new \DateInterval('PT'.random_int(0, $this->randomDelay).'S'));`
	`46`	`+ }`
	`47`	`+`
	`48`	`+ return \DateTimeImmutable::createFromMutable($nextRun);`
`43`	`49`	`}`
`44`	`50`	`}`