diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2015-12-11 17:22:28 +0100 |
---|---|---|
committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2015-12-11 17:22:28 +0100 |
commit | ff84530c0aa72d2a035ce64749687e559a641600 (patch) | |
tree | 23ee8e3356a0757162b1ae48186d289a827a6688 | |
parent | 1a75d086b53d799ac10f77acfb198f2c5ea95621 (diff) | |
download | gitlab-shell-ff84530c0aa72d2a035ce64749687e559a641600.tar.gz |
Remove trailing slashes from gitlab_url
They do not play nice with gitlab-workhorse (or rather Golang net/http
DefaultServemux).
-rw-r--r-- | config.yml.example | 10 | ||||
-rw-r--r-- | lib/gitlab_config.rb | 2 | ||||
-rw-r--r-- | spec/gitlab_config_spec.rb | 6 |
3 files changed, 12 insertions, 6 deletions
diff --git a/config.yml.example b/config.yml.example index e7ecc01..8d19a97 100644 --- a/config.yml.example +++ b/config.yml.example @@ -6,14 +6,14 @@ # GitLab user. git by default user: git -# Url to gitlab instance. Used for api calls. Should end with a slash. -# Default: http://localhost:8080/ +# Url to gitlab instance. Used for api calls. +# Default: http://localhost # You only have to change the default if you have configured Unicorn # to listen on a custom port, or if you have configured Unicorn to # only listen on a Unix domain socket. For Unix domain sockets use -# "http+unix://<urlquoted-path-to-socket>/", e.g. -# "http+unix://%2Fpath%2Fto%2Fsocket/" -gitlab_url: "http://localhost:8080/" +# "http+unix://<urlquoted-path-to-socket>", e.g. +# "http+unix://%2Fpath%2Fto%2Fsocket" +gitlab_url: "http://localhost:8080" # See installation.md#using-https for additional HTTPS configuration details. http_settings: diff --git a/lib/gitlab_config.rb b/lib/gitlab_config.rb index caca176..a8d03f3 100644 --- a/lib/gitlab_config.rb +++ b/lib/gitlab_config.rb @@ -24,7 +24,7 @@ class GitlabConfig end def gitlab_url - @config['gitlab_url'] ||= "http://localhost/" + (@config['gitlab_url'] ||= "http://localhost").sub(%r{/*$}, '') end def http_settings diff --git a/spec/gitlab_config_spec.rb b/spec/gitlab_config_spec.rb index 52fb182..0ce641b 100644 --- a/spec/gitlab_config_spec.rb +++ b/spec/gitlab_config_spec.rb @@ -35,6 +35,12 @@ eos it { should_not be_empty } it { should eq(url) } + + context 'remove trailing slashes' do + before { config.send(:config)['gitlab_url'] = url + '//' } + + it { should eq(url) } + end end describe :audit_usernames do |