diff options
author | William Hilton <wmhilton@gmail.com> | 2020-07-07 18:31:49 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-08 01:31:49 +0300 |
commit | 3a46d1ef58437028b8abea36a42affb6fff4ed0b (patch) | |
tree | 5a48ef0efdef62c351571857a31ec7ad0e820c26 | |
parent | 078d2fbd4c386d0cf37378078443f5f8dbe89f37 (diff) | |
download | gitea-3a46d1ef58437028b8abea36a42affb6fff4ed0b.tar.gz gitea-3a46d1ef58437028b8abea36a42affb6fff4ed0b.zip |
Enable cloning via Git Wire Protocol v2 over HTTP (#12170)
* Enable Git Wire Protocol v2 over HTTP
* use RunInDirTimeoutEnv
* pass $HOME environment variable to git upload-pack
-rw-r--r-- | routers/repo/http.go | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/routers/repo/http.go b/routers/repo/http.go index 0481b46bde..565ee5eff4 100644 --- a/routers/repo/http.go +++ b/routers/repo/http.go @@ -483,6 +483,9 @@ var routes = []route{ {regexp.MustCompile(`(.*?)/objects/pack/pack-[0-9a-f]{40}\.idx$`), "GET", getIdxFile}, } +// one or more key=value pairs separated by colons +var safeGitProtocolHeader = regexp.MustCompile(`^[0-9a-zA-Z]+=[0-9a-zA-Z]+(:[0-9a-zA-Z]+=[0-9a-zA-Z]+)*$`) + func getGitConfig(option, dir string) string { out, err := git.NewCommand("config", option).RunInDir(dir) if err != nil { @@ -553,14 +556,16 @@ func serviceRPC(h serviceHandler, service string) { // set this for allow pre-receive and post-receive execute h.environ = append(h.environ, "SSH_ORIGINAL_COMMAND="+service) + if protocol := h.r.Header.Get("Git-Protocol"); protocol != "" && safeGitProtocolHeader.MatchString(protocol) { + h.environ = append(h.environ, "GIT_PROTOCOL="+protocol) + } + ctx, cancel := gocontext.WithCancel(git.DefaultContext) defer cancel() var stderr bytes.Buffer cmd := exec.CommandContext(ctx, git.GitExecutable, service, "--stateless-rpc", h.dir) cmd.Dir = h.dir - if service == "receive-pack" { - cmd.Env = append(os.Environ(), h.environ...) - } + cmd.Env = append(os.Environ(), h.environ...) cmd.Stdout = h.w cmd.Stdin = reqBody cmd.Stderr = &stderr @@ -610,7 +615,13 @@ func getInfoRefs(h serviceHandler) { h.setHeaderNoCache() if hasAccess(getServiceType(h.r), h, false) { service := getServiceType(h.r) - refs, err := git.NewCommand(service, "--stateless-rpc", "--advertise-refs", ".").RunInDirBytes(h.dir) + + if protocol := h.r.Header.Get("Git-Protocol"); protocol != "" && safeGitProtocolHeader.MatchString(protocol) { + h.environ = append(h.environ, "GIT_PROTOCOL="+protocol) + } + h.environ = append(os.Environ(), h.environ...) + + refs, err := git.NewCommand(service, "--stateless-rpc", "--advertise-refs", ".").RunInDirTimeoutEnv(h.environ, -1, h.dir) if err != nil { log.Error(fmt.Sprintf("%v - %s", err, string(refs))) } |