From eb3b35b9b0cc55fb8464d9b0662e6b94aafc54cc Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Mon, 17 Aug 2020 22:19:56 -0700 Subject: Fix gitlab-shell not handling relative URLs over UNIX sockets From https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/4498#note_397401883, if you specify a relative path such as: ``` external_url 'http://gitlab.example.com/gitlab' ``` gitlab-shell doesn't have a way to pass the `/gitlab` to the host. For example, let's say we have: ``` gitlab_url: "http+unix://%2Fvar%2Fopt%2Fgitlab%2Fgitlab-workhorse%2Fsocket" ``` If we have `/gitlab` as the relative path, how do we specify what is the UNIX socket path and what is the relative path? If we specify: ``` gitlab_url: "http+unix:///var/opt/gitlab/gitlab-workhorse.socket/gitlab ``` This is ambiguous. Is the socket in `/var/opt/gitlab/gitlab-workhorse.socket/gitlab` or in `/var/opt/gitlab/gitlab-workhorse.socket`? To fix this, this merge request adds an optional `gitlab_relative_url_root` config parameter: ``` gitlab_url: "http+unix://%2Fvar%2Fopt%2Fgitlab%2Fgitlab-workhorse%2Fsocket" gitlab_relative_url_root: /gitlab ``` This is only used with UNIX domain sockets to disambiguate the socket and base URL path. If `gitlab_url` uses `http://` or `https://`, then `gitlab_relative_url_root` is ignored. Relates to https://gitlab.com/gitlab-org/gitlab-shell/-/issues/476 --- client/httpsclient_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'client/httpsclient_test.go') diff --git a/client/httpsclient_test.go b/client/httpsclient_test.go index 6c3ae08..1c7435f 100644 --- a/client/httpsclient_test.go +++ b/client/httpsclient_test.go @@ -106,7 +106,7 @@ func setupWithRequests(t *testing.T, caFile, caPath string, selfSigned bool) (*G url, cleanup := testserver.StartHttpsServer(t, requests) - httpClient := NewHTTPClient(url, caFile, caPath, selfSigned, 1) + httpClient := NewHTTPClient(url, "", caFile, caPath, selfSigned, 1) client, err := NewGitlabNetClient("", "", "", httpClient) require.NoError(t, err) -- cgit v1.2.1