diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2016-12-20 13:14:20 +0300 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2016-12-20 13:14:20 +0300 |
commit | deb74f73d9432c90649142cf8333c5cd5d0984ea (patch) | |
tree | e4928ef04afee97dd500b5e609f2290ad2d14bb2 /lib | |
parent | 52278412c7350b8087ef4ffc688c79e4593369cc (diff) | |
parent | 212967aefb55e0792a36e67a881b66c8bd871e9f (diff) |
Merge branch '25848-fix-git-rev-list-env-parsing' into 'master'
Reject blank environment variables in Gitlab::Git::RevList
Closes #25848
See merge request !8189
Diffstat (limited to 'lib')
-rw-r--r-- | lib/gitlab/git/rev_list.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/gitlab/git/rev_list.rb b/lib/gitlab/git/rev_list.rb index 25e9d619697..79dd0cf7df2 100644 --- a/lib/gitlab/git/rev_list.rb +++ b/lib/gitlab/git/rev_list.rb @@ -22,7 +22,7 @@ module Gitlab def valid? environment_variables.all? do |(name, value)| - value.start_with?(project.repository.path_to_repo) + value.to_s.start_with?(project.repository.path_to_repo) end end @@ -35,7 +35,7 @@ module Gitlab end def environment_variables - @environment_variables ||= env.slice(*ALLOWED_VARIABLES) + @environment_variables ||= env.slice(*ALLOWED_VARIABLES).compact end end end |