cron.go: don't log nil error

Issue #294 new
Trek Hopton created an issue

When setting a cron, the log shows “<nil>”, upon inspection this is because an error is being logged unnecessarily.

log.Printf("cron run: setting %s=%q for site=%d: %v", job.Var, job.Data, job.Skey, err)

This is on line 174.

The same thing happens on line 182 for deleting a cron:

log.Printf("cron run: deleting %s for site=%d: %v", job.Var, job.Skey, err)

err shouldn’t be logged here either.

Comments (0)

  1. Log in to comment