Return time.Time{} when cron string is empty (goharbor#17289)

change log level to debug to avoid noise Signed-off-by: stonezdj <[email protected]> Signed-off-by: Stephan Hohn <[email protected]>
mcsage · Feb 16, 2023 · be75dbf · be75dbf
1 parent 647663c
commit be75dbf
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/common/utils/utils.go b/src/common/utils/utils.go
@@ -284,10 +284,13 @@ func FindNamedMatches(regex *regexp.Regexp, str string) map[string]string {
 // the cron string could contain 6 tokens
 // if the cron string is invalid, it returns a zero time
 func NextSchedule(cron string, curTime time.Time) time.Time {
+	if len(cron) == 0 {
+		return time.Time{}
+	}
 	cr := strings.TrimSpace(cron)
 	s, err := CronParser().Parse(cr)
 	if err != nil {
-		log.Errorf("the cron string %v is invalid, error: %v", cron, err)
+		log.Debugf("the cron string %v is invalid, error: %v", cron, err)
 		return time.Time{}
 	}
 	return s.Next(curTime)