docs: update rest api desc

Author: fiftin

api-docs.yml

@@ -713,20 +713,27 @@ definitions:
         type: integer
       status:
         type: string
-      debug:
-        type: boolean
       playbook:
         type: string
       environment:
         type: string
       secret:
         type: string
-      limit:
-        type: string
       git_branch:
         type:
           - string
           - 'null'
+      params:
+        type: object
+        properties:
+          debug:
+            type: boolean
+          dry_run:
+            type: boolean
+          diff:
+            type: boolean
+          limit:
+            type: string
       message:
         type: string
 

db/Task.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	"strings"
 	"time"
 
 	"github.com/semaphoreui/semaphore/pkg/tz"
@@ -74,6 +75,9 @@ type Task struct {
 	InventoryID *int `db:"inventory_id" json:"inventory_id,omitempty"`
 
 	Params MapStringAnyField `db:"params" json:"params,omitempty"`
+
+	// Limit is deprecated, use Params.Limit instead
+	Limit string `db:"hosts_limit" json:"limit"`
 }
 
 func (task *Task) FillParams(target interface{}) (err error) {
@@ -85,9 +89,25 @@ func (task *Task) FillParams(target interface{}) (err error) {
 	return
 }
 
+// PreInsert is a hook which is called before inserting task into database.
+// Called directly in BoltDB implementation.
 func (task *Task) PreInsert(gorp.SqlExecutor) error {
 	task.Created = tz.In(task.Created)
 
+	if _, ok := task.Params["limit"]; !ok {
+		if task.Params == nil {
+			task.Params = make(MapStringAnyField)
+		}
+
+		limits := strings.Split(task.Limit, ",")
+
+		for i := range limits {
+			limits[i] = strings.TrimSpace(limits[i])
+		}
+
+		task.Params["limit"] = limits
+	}
+
 	return nil
 }
 

db/bolt/task.go

@@ -2,7 +2,6 @@ package bolt
 
 import (
 	"github.com/semaphoreui/semaphore/db"
-	"github.com/semaphoreui/semaphore/pkg/tz"
 	"go.etcd.io/bbolt"
 )
 
@@ -90,7 +89,11 @@ func (d *BoltDb) clearTasks(projectID int, templateID int, maxTasks int) {
 }
 
 func (d *BoltDb) CreateTask(task db.Task, maxTasks int) (newTask db.Task, err error) {
-	task.Created = tz.Now()
+	err = task.PreInsert(nil)
+	if err != nil {
+		return
+	}
+
 	task.ID = 0
 	res, err := d.createObject(0, db.TaskProps, task)
 	if err != nil {

openapi.yml

@@ -1556,22 +1556,26 @@ paths:
               properties:
                 template_id:
                   type: integer
-                debug:
-                  type: boolean
-                dry_run:
-                  type: boolean
-                diff:
-                  type: boolean
                 playbook:
                   type: string
                 environment:
                   type: string
-                limit:
-                  type: string
                 git_branch:
                   type: string
                 message:
                   type: string
+                params:
+                  type: object
+                  properties:
+                    debug:
+                      type: boolean
+                    dry_run:
+                      type: boolean
+                    diff:
+                      type: boolean
+                    limit:
+                      type: string
+
         required: true
       responses:
         '201':