Remove trailing slashes from gitlab_url

They do not play nice with gitlab-workhorse (or rather Golang net/http
DefaultServemux).

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