Git client will follow 301 but 307 (go-gitea#34005)

lunny · wxiaoguang · Gusted · commit 50f8563c6798 · 2025-04-01T02:28:02.000+02:00
Fix go-gitea#28460 --------- Co-authored-by: wxiaoguang <wxiaoguang@gmail.com> (cherry picked from commit 356b707)
diff --git a/services/context/repo.go b/services/context/repo.go
@@ -361,7 +361,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
`@@ -361,7 +361,9 @@ func RedirectToRepo(ctx *Base, redirectRepoID int64) {`
`361`	`361`	`if ctx.Req.URL.RawQuery != "" {`
`362`	`362`	`redirectPath += "?" + ctx.Req.URL.RawQuery`
`363`	`363`	`}`
`364`		`- ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusTemporaryRedirect)`
	`364`	`+ // Git client needs a 301 redirect by default to follow the new location`
	`365`	`+ // It's not documentated in git documentation, but it's the behavior of git client`
	`366`	`+ ctx.Redirect(path.Join(setting.AppSubURL, redirectPath), http.StatusMovedPermanently)`
`365`	`367`	`}`
`366`	`368`
`367`	`369`	`func repoAssignment(ctx Context, repo repo_model.Repository) {`