if repo does not exist, show a 404 not a 500 (#5900)
This commit is contained in:
parent
a967cf9e99
commit
8b5f6ced22
1 changed files with 4 additions and 1 deletions
|
@ -287,7 +287,10 @@ func Issues(ctx *context.Context) {
|
||||||
if repoID > 0 {
|
if repoID > 0 {
|
||||||
if _, ok := showReposMap[repoID]; !ok {
|
if _, ok := showReposMap[repoID]; !ok {
|
||||||
repo, err := models.GetRepositoryByID(repoID)
|
repo, err := models.GetRepositoryByID(repoID)
|
||||||
if err != nil {
|
if models.IsErrRepoNotExist(err) {
|
||||||
|
ctx.NotFound("GetRepositoryByID", err)
|
||||||
|
return
|
||||||
|
} else if err != nil {
|
||||||
ctx.ServerError("GetRepositoryByID", fmt.Errorf("[%d]%v", repoID, err))
|
ctx.ServerError("GetRepositoryByID", fmt.Errorf("[%d]%v", repoID, err))
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
Reference in a new issue