summaryrefslogtreecommitdiff
path: root/client/httpsclient_test.go
diff options
context:
space:
mode:
authorStan Hu <stanhu@gmail.com>2020-08-17 22:19:56 -0700
committerStan Hu <stanhu@gmail.com>2020-08-20 16:54:36 -0700
commiteb3b35b9b0cc55fb8464d9b0662e6b94aafc54cc (patch)
treef25886b6a225f108c67c423dcbe13f027d4a18c1 /client/httpsclient_test.go
parentfa730d2f859671f54c6f88bf2551fc771a1a5e6a (diff)
downloadgitlab-shell-eb3b35b9b0cc55fb8464d9b0662e6b94aafc54cc.tar.gz
Fix gitlab-shell not handling relative URLs over UNIX socketssh-fix-unix-relative-url-access
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
Diffstat (limited to 'client/httpsclient_test.go')
-rw-r--r--client/httpsclient_test.go2
1 files changed, 1 insertions, 1 deletions
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)