diff --git a/services/context/repo.go b/services/context/repo.go index cabdd1d158..4dc8539270 100644 --- a/services/context/repo.go +++ b/services/context/repo.go @@ -400,25 +400,25 @@ func repoAssignment(ctx *Context, repo *repo_model.Repository) { if ctx.Doer != nil { ctx.Data["AllowCreateDevcontainer"] = ctx.Doer.AllowCreateDevcontainer ctx.Data["AllowCreateActRunner"] = ctx.Doer.AllowCreateActRunner - } else { query := ctx.Req.URL.Query() userID := query.Get("user") - userNum, err := strconv.ParseInt(userID, 10, 64) - if err != nil { - return - } - u, err := user_model.GetUserByID(ctx, userNum) - if err != nil { - if user_model.IsErrUserNotExist(err) { - ctx.Redirect(setting.AppSubURL + "/-/admin/users") - } else { - ctx.ServerError("GetUserByID", err) + if userID != "" { + userNum, err := strconv.ParseInt(userID, 10, 64) + if err == nil { + u, err := user_model.GetUserByID(ctx, userNum) + if err != nil { + if user_model.IsErrUserNotExist(err) { + ctx.Redirect(setting.AppSubURL + "/-/admin/users") + } else { + ctx.ServerError("GetUserByID", err) + } + return + } + ctx.Data["AllowCreateDevcontainer"] = u.AllowCreateDevcontainer + ctx.Data["AllowCreateActRunner"] = u.AllowCreateActRunner } - return } - ctx.Data["AllowCreateDevcontainer"] = u.AllowCreateDevcontainer - ctx.Data["AllowCreateActRunner"] = u.AllowCreateActRunner } if repo.IsMirror {