From 22e1bb64660f4ce8c75dec635b613716fe276c9f Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Mon, 12 Feb 2024 02:37:43 -0500 Subject: [PATCH] feat: add more context to errors in fetching cron entries --- plugins/cron/cron.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/cron/cron.go b/plugins/cron/cron.go index de65de156..88a3fa89b 100644 --- a/plugins/cron/cron.go +++ b/plugins/cron/cron.go @@ -59,7 +59,7 @@ func FetchCronEntries(appName string) ([]TemplateCommand, error) { commands := []TemplateCommand{} appJSON, err := appjson.GetAppJSON(appName) if err != nil { - return commands, err + return commands, fmt.Errorf("Unable to fetch app.json for app %s: %s", appName, err.Error()) } if appJSON.Cron == nil { @@ -70,7 +70,7 @@ func FetchCronEntries(appName string) ([]TemplateCommand, error) { parser := cronparser.NewParser(cronparser.Minute | cronparser.Hour | cronparser.Dom | cronparser.Month | cronparser.Dow | cronparser.Descriptor) _, err := parser.Parse(c.Schedule) if err != nil { - return commands, err + return commands, fmt.Errorf("Invalid cron schedule %s: %s", c.Schedule, err.Error()) } commands = append(commands, TemplateCommand{