diff options
author | Stan Hu <stanhu@gmail.com> | 2019-01-29 07:49:59 -0800 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-01-29 08:08:14 -0800 |
commit | d3a7d925b931348c16a3abb7a8d5c72aa7b05ab7 (patch) | |
tree | 019160858ed518ca8d35d24152dc6f2b8f586959 | |
parent | b9e17d11d3e6821693a336217c63c6486ce86273 (diff) | |
download | gitlab-ce-d3a7d925b931348c16a3abb7a8d5c72aa7b05ab7.tar.gz |
Fix uninitialized constant with GitLab Pages deploy
pages:deploy step was failing with the following error:
```
unitialized constant SafeZip::Extract::Zip
```
Since license_finder already pulls in rubyzip, we can make it
a required gem. We also use the scope operator to make the reference to
Zip::File explicit.
-rw-r--r-- | Gemfile | 2 | ||||
-rw-r--r-- | changelogs/unreleased/sh-fix-pages-zip-constant.yml | 5 | ||||
-rw-r--r-- | lib/safe_zip/extract.rb | 2 |
3 files changed, 7 insertions, 2 deletions
@@ -77,7 +77,7 @@ gem 'u2f', '~> 0.2.1' # GitLab Pages gem 'validates_hostname', '~> 1.0.6' -gem 'rubyzip', '~> 1.2.2', require: false +gem 'rubyzip', '~> 1.2.2' # Browser detection gem 'browser', '~> 2.5' diff --git a/changelogs/unreleased/sh-fix-pages-zip-constant.yml b/changelogs/unreleased/sh-fix-pages-zip-constant.yml new file mode 100644 index 00000000000..fcd8aa45825 --- /dev/null +++ b/changelogs/unreleased/sh-fix-pages-zip-constant.yml @@ -0,0 +1,5 @@ +--- +title: Fix uninitialized constant with GitLab Pages +merge_request: +author: +type: fixed diff --git a/lib/safe_zip/extract.rb b/lib/safe_zip/extract.rb index 3bd4935ef34..679c021c730 100644 --- a/lib/safe_zip/extract.rb +++ b/lib/safe_zip/extract.rb @@ -29,7 +29,7 @@ module SafeZip private def extract_with_ruby_zip(params) - Zip::File.open(archive_path) do |zip_archive| + ::Zip::File.open(archive_path) do |zip_archive| # Extract all files in the following order: # 1. Directories first, # 2. Files next, |