Merge branch 'remove-trailing-slashes' into 'master'

Remove trailing slashes from gitlab_url

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

See merge request !35

CHANGELOG

@@ -1,3 +1,6 @@
+v2.6.9
+  - Remove trailing slashes from gitlab_url
+
 v2.6.8
   - Revert git-lfs-authenticate command from white list
 

VERSION

@@ -1 +1 @@
-2.6.8
+2.6.9

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:8080
 # 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:

lib/gitlab_config.rb

@@ -24,7 +24,7 @@ def secret_file
   end
 
   def gitlab_url
-    @config['gitlab_url'] ||= "http://localhost/"
+    (@config['gitlab_url'] ||= "http://localhost:8080").sub(%r{/*$}, '')
   end
 
   def http_settings

spec/gitlab_config_spec.rb

@@ -35,6 +35,12 @@
 
     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