From a9d547f55b5e929e902e4d3d72d13d1f62f57741 Mon Sep 17 00:00:00 2001 From: Giteabot Date: Wed, 4 Oct 2023 22:41:57 +0800 Subject: [PATCH] When comparing with an non-exist repository, return 404 but 500 (#27437) (#27442) Backport #27437 by @lunny Co-authored-by: Lunny Xiao --- routers/web/repo/compare.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/routers/web/repo/compare.go b/routers/web/repo/compare.go index ecc8e66702..9af1d60c29 100644 --- a/routers/web/repo/compare.go +++ b/routers/web/repo/compare.go @@ -252,7 +252,6 @@ func ParseCompareInfo(ctx *context.Context) *CompareInfo { isSameRepo = true ci.HeadUser = ctx.Repo.Owner ci.HeadBranch = headInfos[0] - } else if len(headInfos) == 2 { headInfosSplit := strings.Split(headInfos[0], "/") if len(headInfosSplit) == 1 { @@ -407,6 +406,9 @@ func ParseCompareInfo(ctx *context.Context) *CompareInfo { return nil } defer ci.HeadGitRepo.Close() + } else { + ctx.NotFound("ParseCompareInfo", nil) + return nil } ctx.Data["HeadRepo"] = ci.HeadRepo