Git client will follow 301 but 307 (#34005) (#34010)

GiteaBot · lunny · wxiaoguang · web-flow · commit 4b3400bd9c77 · 2025-03-25T15:56:44.000+08:00
Backport #34005 by lunny Fix #28460 Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com> Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
diff --git a/services/context/repo.go b/services/context/repo.go
@@ -356,7 +356,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
`@@ -356,7 +356,9 @@ func RedirectToRepo(ctx *Base, redirectRepoID int64) {`
`356`	`356`	`if ctx.Req.URL.RawQuery != "" {`
`357`	`357`	`redirectPath += "?" + ctx.Req.URL.RawQuery`
`358`	`358`	`}`
`359`		`- ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusTemporaryRedirect)`
	`359`	`+ // Git client needs a 301 redirect by default to follow the new location`
	`360`	`+ // It's not documentated in git documentation, but it's the behavior of git client`
	`361`	`+ ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusMovedPermanently)`
`360`	`362`	`}`
`361`	`363`
`362`	`364`	`func repoAssignment(ctx Context, repo repo_model.Repository) {`