[PATCH v5 2/9] connect: only match the host with core.gitProxy
- From
Mike Hommey <mh@glandium.org>
- Date
- May 16, 2016, 00:07 UTC
- Message-ID
- <20160516000740.19042-3-mh@glandium.org>
- In-Reply-To
- <20160516000740.19042-1-mh@glandium.org>
Currently, core.gitProxy doesn't actually match purely on domain names as documented: it also matches ports.
So a core.gitProxy value like "script for kernel.org" doesn't make the script called for an url like git://kernel.org:port/path, while it is called for git://kernel.org/path.
This per-port behavior seems like an oversight rather than a deliberate choice, so, make git://kernel.org:port/path call the gitProxy script in the case described above.
Signed-off-by: Mike Hommey <mh@glandium.org> --- connect.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/connect.c b/connect.c index d3448c2..2a08318 100644 --- a/connect.c +++ b/connect.c @@ -706,7 +706,7 @@ struct child_process *git_connect(int fd[2], const char *url, /* These underlying connection commands die() if they * cannot connect. */ - if (git_use_proxy(hostandport)) + if (git_use_proxy(host)) conn = git_proxy_connect(fd, host, port); else git_tcp_connect(fd, host, port, flags);
-- 2.8.2.411.ga331486