From 26389f0482e15efcfdec9569d3c3f0f002d9ed6e Mon Sep 17 00:00:00 2001 From: Sipke Schoorstra Date: Wed, 29 May 2024 23:49:40 +0200 Subject: [PATCH] Add validation for empty cron expression This commit adds validation for cron expressions in the DefaultTriggerScheduler class. The system now checks if the cron expression provided is empty and issues a warning if that's the case. This prevents attempts to schedule triggers with an empty cron expression, which would fail. --- .../Elsa.Scheduling/Services/DefaultTriggerScheduler.cs | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/modules/Elsa.Scheduling/Services/DefaultTriggerScheduler.cs b/src/modules/Elsa.Scheduling/Services/DefaultTriggerScheduler.cs index 0389e2f78..2426ddb31 100644 --- a/src/modules/Elsa.Scheduling/Services/DefaultTriggerScheduler.cs +++ b/src/modules/Elsa.Scheduling/Services/DefaultTriggerScheduler.cs @@ -71,6 +71,13 @@ public class DefaultTriggerScheduler : ITriggerScheduler { var payload = trigger.GetPayload(); var cronExpression = payload.CronExpression; + + if (string.IsNullOrWhiteSpace(cronExpression)) + { + _logger.LogWarning("Cron expression is empty. TriggerId: {TriggerId}. Skipping scheduling of this trigger", trigger.Id); + continue; + } + var input = new { CronExpression = cronExpression }.ToDictionary(); var request = new DispatchWorkflowDefinitionRequest {