remove redundant handlerutil.Redirect

sentriz · Apr 28, 2024 · 559c910 · 559c910
1 parent 6ba342c
commit 559c910
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/cmd/gonic/gonic.go b/cmd/gonic/gonic.go
@@ -273,7 +273,7 @@ func main() {
 	mux.Handle("/admin/", http.StripPrefix("/admin", chain(ctrlAdmin)))
 	mux.Handle("/rest/", http.StripPrefix("/rest", chain(trim(ctrlSubsonic))))
 	mux.Handle("/ping", chain(handlerutil.Message("ok")))
-	mux.Handle("/", chain(handlerutil.Redirect(resolveProxyPath("/admin/home"))))
+	mux.Handle("/", chain(http.RedirectHandler(resolveProxyPath("/admin/home"), http.StatusSeeOther)))
 
 	if *confExpvar {
 		mux.Handle("/debug/vars", expvar.Handler())

diff --git a/handlerutil/handlerutil.go b/handlerutil/handlerutil.go
@@ -56,12 +56,6 @@ func BasicCORS(next http.Handler) http.Handler {
 	})
 }
 
-func Redirect(to string) http.Handler {
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		http.Redirect(w, r, to, http.StatusSeeOther)
-	})
-}
-
 func Message(message string) http.Handler {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		fmt.Fprintln(w, message)