Mirrors/gitea
0
0
Fork 0
mirror of https://github.com/go-gitea/gitea.git synced 2026-08-04 04:22:15 +02:00
Code Issues Activity

fix abuses

Browse Source
This commit is contained in:
wxiaoguang
2026-06-06 21:00:14 +08:00
parent b1f72e401a
commit f431889cd9
6 changed files with 15 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
routers/api/v1/admin/user.go
+2 -4
View File
@@ -97,14 +97,12 @@ func CreateUser(ctx *context.APIContext) {
if u.LoginType == auth.Plain {
if len(form.Password) < setting.MinPasswordLength {
err := errors.New("PasswordIsRequired")
ctx.APIError(http.StatusBadRequest, err.Error())
ctx.APIError(http.StatusBadRequest, "PasswordIsRequired")
return
}
if !password.IsComplexEnough(form.Password) {
err := errors.New("PasswordComplexity")
ctx.APIError(http.StatusBadRequest, err.Error())
ctx.APIError(http.StatusBadRequest, "PasswordComplexity")
return
}
routers/api/v1/org/action.go
+1 -1
View File
@@ -387,7 +387,7 @@ func (Action) CreateVariable(ctx *context.APIContext) {
return
}
if v != nil && v.ID > 0 {
ctx.APIError(http.StatusConflict, util.NewAlreadyExistErrorf("variable name %s already exists", variableName).Error())
ctx.APIError(http.StatusConflict, "variable name already exists")
return
}
routers/api/v1/repo/action.go
+3 -3
View File
@@ -361,7 +361,7 @@ func (Action) CreateVariable(ctx *context.APIContext) {
return
}
if v != nil && v.ID > 0 {
ctx.APIError(http.StatusConflict, util.NewAlreadyExistErrorf("variable name %s already exists", variableName).Error())
ctx.APIError(http.StatusConflict, "variable name already exists")
return
}
@@ -1062,7 +1062,7 @@ func ActionsDispatchWorkflow(ctx *context.APIContext) {
workflowID := ctx.PathParam("workflow_id")
opt := web.GetForm(ctx).(*api.CreateActionWorkflowDispatch)
if opt.Ref == "" {
ctx.APIError(http.StatusUnprocessableEntity, util.NewInvalidArgumentErrorf("ref is required parameter").Error())
ctx.APIError(http.StatusUnprocessableEntity, "ref is required parameter")
return
}
@@ -1560,7 +1560,7 @@ func ListWorkflowRunJobs(ctx *context.APIContext) {
// Avoid the list all jobs functionality for this api route to be used with a runID == 0.
if runID <= 0 {
ctx.APIError(http.StatusBadRequest, util.NewInvalidArgumentErrorf("runID must be a positive integer").Error())
ctx.APIError(http.StatusBadRequest, "runID must be a positive integer")
return
}
routers/api/v1/repo/mirror.go
+7 -2
View File
@@ -5,6 +5,7 @@ package repo
import (
"errors"
"fmt"
"net/http"
"strings"
"time"
@@ -344,8 +345,12 @@ func CreatePushMirror(ctx *context.APIContext, mirrorOption *api.CreatePushMirro
repo := ctx.Repo.Repository
interval, err := time.ParseDuration(mirrorOption.Interval)
if err != nil || (interval != 0 && interval < setting.Mirror.MinInterval) {
ctx.APIError(http.StatusBadRequest, err.Error())
if err != nil {
ctx.APIError(http.StatusBadRequest, fmt.Sprintf("invalid interval: %v", err))
return
}
if interval != 0 && interval < setting.Mirror.MinInterval {
ctx.APIError(http.StatusBadRequest, fmt.Sprintf("interval is shorter than minimum %v", setting.Mirror.MinInterval.String()))
return
}
routers/api/v1/repo/release.go
+1 -2
View File
@@ -4,7 +4,6 @@
package repo
import (
"fmt"
"net/http"
auth_model "gitea.dev/models/auth"
@@ -276,7 +275,7 @@ func CreateRelease(ctx *context.APIContext) {
} else if release_service.IsErrProtectedTagName(err) {
ctx.APIError(http.StatusUnprocessableEntity, err.Error())
} else if git.IsErrNotExist(err) {
ctx.APIError(http.StatusNotFound, fmt.Sprintf("target \"%v\" not found: %v", rel.Target, err))
ctx.APIError(http.StatusNotFound, "target not found")
} else {
ctx.APIErrorInternal(err)
}
routers/api/v1/user/action.go
+1 -1
View File
@@ -148,7 +148,7 @@ func CreateVariable(ctx *context.APIContext) {
return
}
if v != nil && v.ID > 0 {
ctx.APIError(http.StatusConflict, util.NewAlreadyExistErrorf("variable name %s already exists", variableName).Error())
ctx.APIError(http.StatusConflict, "variable name already exists")
return
}