fix: allow whole repository url on form

caarlos0 · caarlos0 · commit 862be19e467d · 2021-08-04T23:25:54.000-03:00
Signed-off-by: Carlos Alexandro Becker &lt;caarlos0@gmail.com&gt;
diff --git a/controller/index.go b/controller/index.go
@@ -5,6 +5,7 @@ import (
 	"io"
 	"io/fs"
 	"net/http"
+	"strings"
 
 	"github.com/caarlos0/httperr"
 )
@@ -17,7 +18,8 @@ func Index(fsys fs.FS, version string) http.Handler {
 
 func HandleForm(fsys fs.FS) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
-		http.Redirect(w, r, "/"+r.FormValue("repository"), http.StatusSeeOther)
+		repo := strings.TrimPrefix(r.FormValue("repository"), "https://github.com/")
+		http.Redirect(w, r, repo, http.StatusSeeOther)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ import (`
`5`	`5`	`"io"`
`6`	`6`	`"io/fs"`
`7`	`7`	`"net/http"`
	`8`	`+ "strings"`
`8`	`9`
`9`	`10`	`"github.com/caarlos0/httperr"`
`10`	`11`	`)`
`@@ -17,7 +18,8 @@ func Index(fsys fs.FS, version string) http.Handler {`
`17`	`18`
`18`	`19`	`func HandleForm(fsys fs.FS) http.HandlerFunc {`
`19`	`20`	`return func(w http.ResponseWriter, r *http.Request) {`
`20`		`- http.Redirect(w, r, "/"+r.FormValue("repository"), http.StatusSeeOther)`
	`21`	`+ repo := strings.TrimPrefix(r.FormValue("repository"), "https://github.com/")`
	`22`	`+ http.Redirect(w, r, repo, http.StatusSeeOther)`
`21`	`23`	`}`
`22`	`24`	`}`
`23`	`25`