Git client will follow 301 but 307 (#34005)

lunny · wxiaoguang · web-flow · commit 356b707dde00 · 2025-03-25T07:20:08.000Z
Fix #28460 --------- Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
diff --git a/services/context/repo.go b/services/context/repo.go
@@ -328,7 +328,9 @@ func RedirectToRepo(ctx *Base, redirectRepoID int64) {
 	if ctx.Req.URL.RawQuery != "" {
 		redirectPath += "?" + ctx.Req.URL.RawQuery
 	}
-	ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusTemporaryRedirect)
+	// Git client needs a 301 redirect by default to follow the new location
+	// It's not documentated in git documentation, but it's the behavior of git client
+	ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusMovedPermanently)
 }
 
 func repoAssignment(ctx *Context, repo *repo_model.Repository) {

Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,9 @@ func RedirectToRepo(ctx *Base, redirectRepoID int64) {`
`328`	`328`	`if ctx.Req.URL.RawQuery != "" {`
`329`	`329`	`redirectPath += "?" + ctx.Req.URL.RawQuery`
`330`	`330`	`}`
`331`		`- ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusTemporaryRedirect)`
	`331`	`+ // Git client needs a 301 redirect by default to follow the new location`
	`332`	`+ // It's not documentated in git documentation, but it's the behavior of git client`
	`333`	`+ ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusMovedPermanently)`
`332`	`334`	`}`
`333`	`335`
`334`	`336`	`func repoAssignment(ctx Context, repo repo_model.Repository) {`