fix: small bug fixes

Some small bug fixes that came up during code review

- `handleError` returns wrong error variable in
  `swarm/pgbackrest_restore.go`
- empty `Statuses` slice filters out all tasks in `task/service.go`
- error logged unconditionally regardless of `logTaskEvent` outcome in
  `workflows/common.go`

Author: jason-lynch

server/internal/orchestrator/swarm/pgbackrest_restore.go

@@ -91,9 +91,13 @@ func (p *PgBackRestRestore) Create(ctx context.Context, rc *resource.Context) er
 	}
 
 	t, err := p.startTask(ctx, taskSvc)
+	if err != nil {
+		return err
+	}
+
 	handleError := func(cause error) error {
 		p.failTask(logger, taskSvc, t, cause)
-		return err
+		return cause
 	}
 
 	svcResource, err := resource.FromContext[*PostgresService](rc, PostgresServiceResourceIdentifier(p.InstanceID))

server/internal/task/service.go

@@ -275,7 +275,7 @@ func matchesFilters(task *Task, opts TaskListOptions) bool {
 	if opts.Type != "" && task.Type != opts.Type {
 		return false
 	}
-	if !slices.Contains(opts.Statuses, task.Status) {
+	if len(opts.Statuses) > 0 && !slices.Contains(opts.Statuses, task.Status) {
 		return false
 	}
 	if opts.NodeName != "" && (task == nil || task.NodeName != opts.NodeName) {

server/internal/workflows/common.go

@@ -191,8 +191,9 @@ func (w *Workflows) cancelTask(
 		Message: "task successfully canceled",
 		Fields:  map[string]any{"status": "canceled"},
 	})
-	logger.With("error", err).Error("failed to log task event")
-
+	if err != nil {
+		logger.With("error", err).Error("failed to log task event")
+	}
 }
 
 func (w *Workflows) getNodeResources(