Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-11-26 20:26:13 +01:00
Code Issues Projects Releases Wiki Activity

Update validation and error message per comments

Browse Source
This commit is contained in:
wang yan 2019-01-17 15:33:05 +08:00
parent 4cde11892a
commit 903e15235e
2 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/common/models/robot.go
View File

@ -53,11 +53,7 @@ type RobotReq struct {
// Valid put request validation
func (rq *RobotReq) Valid(v *validation.Validation) {
switch rq.Disabled {
case true, false:
default:
v.SetError("disabled", "must be in [true, false]")
}
// ToDo: add validation for access info.
}
// RobotRep ...

10
src/core/api/robot.go
View File

@ -45,13 +45,13 @@ func (r *RobotAPI) Prepare() {
pid, err := r.GetInt64FromPath(":pid")
if err != nil || pid <= 0 {
text := "invalid project ID: "
var errMsg string
if err != nil {
text += err.Error()
errMsg = "failed to get project ID " + err.Error()
} else {
text += fmt.Sprintf("%d", pid)
errMsg = "invalid project ID: " + fmt.Sprintf("%d", pid)
}
r.HandleBadRequest(text)
r.HandleBadRequest(errMsg)
return
}
project, err := r.ProjectMgr.Get(pid)
@ -137,7 +137,7 @@ func (r *RobotAPI) List() {
count, err := dao.CountRobot(&query)
if err != nil {
r.HandleInternalServerError(fmt.Sprintf("failed to count robots %v", err))
r.HandleInternalServerError(fmt.Sprintf("failed to list robots on project: %d, %v", r.project.ProjectID, err))
return
}
query.Page, query.Size = r.GetPaginationParams()