From 80098bd7526fed84cfb4fa48272be71541b37205 Mon Sep 17 00:00:00 2001 From: Lauris BH Date: Wed, 30 Jan 2019 23:04:19 +0200 Subject: Fix go-get URL generation (#5905) --- modules/context/context.go | 2 +- modules/context/repo.go | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) (limited to 'modules/context') diff --git a/modules/context/context.go b/modules/context/context.go index b9736e56ef..421d1d5f4f 100644 --- a/modules/context/context.go +++ b/modules/context/context.go @@ -209,7 +209,7 @@ func Contexter() macaron.Handler { if err == nil && len(repo.DefaultBranch) > 0 { branchName = repo.DefaultBranch } - prefix := setting.AppURL + path.Join(ownerName, repoName, "src", "branch", branchName) + prefix := setting.AppURL + path.Join(url.QueryEscape(ownerName), url.QueryEscape(repoName), "src", "branch", branchName) c.Header().Set("Content-Type", "text/html") c.WriteHeader(http.StatusOK) c.Write([]byte(com.Expand(` diff --git a/modules/context/repo.go b/modules/context/repo.go index 310aa886d3..f9e366ed48 100644 --- a/modules/context/repo.go +++ b/modules/context/repo.go @@ -8,6 +8,7 @@ package context import ( "fmt" "io/ioutil" + "net/url" "path" "strings" @@ -171,7 +172,7 @@ func RetrieveBaseRepo(ctx *Context, repo *models.Repository) { // ComposeGoGetImport returns go-get-import meta content. func ComposeGoGetImport(owner, repo string) string { - return path.Join(setting.Domain, setting.AppSubURL, owner, repo) + return path.Join(setting.Domain, setting.AppSubURL, url.QueryEscape(owner), url.QueryEscape(repo)) } // EarlyResponseForGoGetMeta responses appropriate go-get meta with status 200 -- cgit v1.2.3