don't record error when request a non-exist user (#15819)

lunny · zeripath · techknowlogick · web-flow · commit d37a89ed059a · 2021-05-10T13:43:55.000-04:00
* don't record error when request a non-exist user

* Update routers/repo/http.go

Co-authored-by: zeripath &lt;art27@cantab.net&gt;

Co-authored-by: zeripath &lt;art27@cantab.net&gt;
Co-authored-by: techknowlogick &lt;techknowlogick@gitea.io&gt;
diff --git a/routers/repo/http.go b/routers/repo/http.go
@@ -117,7 +117,7 @@ func httpBase(ctx *context.Context) (h *serviceHandler) {
 			if redirectUserID, err := models.LookupUserRedirect(username); err == nil {
 				context.RedirectToUser(ctx, username, redirectUserID)
 			} else {
-				ctx.NotFound("GetUserByName", err)
+				ctx.NotFound(fmt.Sprintf("User %s does not exist", username), nil)
 			}
 		} else {
 			ctx.ServerError("GetUserByName", err)

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ func httpBase(ctx context.Context) (h serviceHandler) {`
`117`	`117`	`if redirectUserID, err := models.LookupUserRedirect(username); err == nil {`
`118`	`118`	`context.RedirectToUser(ctx, username, redirectUserID)`
`119`	`119`	`} else {`
`120`		`- ctx.NotFound("GetUserByName", err)`
	`120`	`+ ctx.NotFound(fmt.Sprintf("User %s does not exist", username), nil)`
`121`	`121`	`}`
`122`	`122`	`} else {`
`123`	`123`	`ctx.ServerError("GetUserByName", err)`