Display friendly error message (#29105)
`ctx.Error` only displays the text but `ctx.ServerError` renders the usual error page.
This commit is contained in:
		| @@ -61,17 +61,17 @@ func List(ctx *context.Context) { | ||||
|  | ||||
| 	var workflows []Workflow | ||||
| 	if empty, err := ctx.Repo.GitRepo.IsEmpty(); err != nil { | ||||
| 		ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 		ctx.ServerError("IsEmpty", err) | ||||
| 		return | ||||
| 	} else if !empty { | ||||
| 		commit, err := ctx.Repo.GitRepo.GetBranchCommit(ctx.Repo.Repository.DefaultBranch) | ||||
| 		if err != nil { | ||||
| 			ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 			ctx.ServerError("GetBranchCommit", err) | ||||
| 			return | ||||
| 		} | ||||
| 		entries, err := actions.ListWorkflows(commit) | ||||
| 		if err != nil { | ||||
| 			ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 			ctx.ServerError("ListWorkflows", err) | ||||
| 			return | ||||
| 		} | ||||
|  | ||||
| @@ -95,7 +95,7 @@ func List(ctx *context.Context) { | ||||
| 			workflow := Workflow{Entry: *entry} | ||||
| 			content, err := actions.GetContentFromEntry(entry) | ||||
| 			if err != nil { | ||||
| 				ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 				ctx.ServerError("GetContentFromEntry", err) | ||||
| 				return | ||||
| 			} | ||||
| 			wf, err := model.ReadWorkflow(bytes.NewReader(content)) | ||||
| @@ -172,7 +172,7 @@ func List(ctx *context.Context) { | ||||
|  | ||||
| 	runs, total, err := db.FindAndCount[actions_model.ActionRun](ctx, opts) | ||||
| 	if err != nil { | ||||
| 		ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 		ctx.ServerError("FindAndCount", err) | ||||
| 		return | ||||
| 	} | ||||
|  | ||||
| @@ -181,7 +181,7 @@ func List(ctx *context.Context) { | ||||
| 	} | ||||
|  | ||||
| 	if err := actions_model.RunList(runs).LoadTriggerUser(ctx); err != nil { | ||||
| 		ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 		ctx.ServerError("LoadTriggerUser", err) | ||||
| 		return | ||||
| 	} | ||||
|  | ||||
| @@ -189,7 +189,7 @@ func List(ctx *context.Context) { | ||||
|  | ||||
| 	actors, err := actions_model.GetActors(ctx, ctx.Repo.Repository.ID) | ||||
| 	if err != nil { | ||||
| 		ctx.Error(http.StatusInternalServerError, err.Error()) | ||||
| 		ctx.ServerError("GetActors", err) | ||||
| 		return | ||||
| 	} | ||||
| 	ctx.Data["Actors"] = repo.MakeSelfOnTop(ctx.Doer, actors) | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 KN4CK3R
					KN4CK3R