diff options
author | James Lopez <james@jameslopez.es> | 2018-12-05 09:52:56 +0100 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2018-12-17 16:14:35 +0100 |
commit | 43cc0d5a4ad4464901f700c0a46dbd304c1005c8 (patch) | |
tree | 7b3edb57d64f7e975a8c4bc1859e0502da31efe4 /lib/gitlab/import_export | |
parent | 574ae4c7aaf34347839c9a44648bfcfea5a1e434 (diff) | |
download | gitlab-ce-43cc0d5a4ad4464901f700c0a46dbd304c1005c8.tar.gz |
Fix persistent symlink in project import
- Fix permissions after untar is done
- Refactor command line util
Diffstat (limited to 'lib/gitlab/import_export')
-rw-r--r-- | lib/gitlab/import_export/command_line_util.rb | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/gitlab/import_export/command_line_util.rb b/lib/gitlab/import_export/command_line_util.rb index c9e2a6a78d9..bdecff0931c 100644 --- a/lib/gitlab/import_export/command_line_util.rb +++ b/lib/gitlab/import_export/command_line_util.rb @@ -3,7 +3,8 @@ module Gitlab module ImportExport module CommandLineUtil - DEFAULT_MODE = 0700 + UNTAR_MASK = 'u+rwX,go+rX,go-w' + DEFAULT_DIR_MODE = 0700 def tar_czf(archive:, dir:) tar_with_options(archive: archive, dir: dir, options: 'czf') @@ -14,8 +15,8 @@ module Gitlab end def mkdir_p(path) - FileUtils.mkdir_p(path, mode: DEFAULT_MODE) - FileUtils.chmod(DEFAULT_MODE, path) + FileUtils.mkdir_p(path, mode: DEFAULT_DIR_MODE) + FileUtils.chmod(DEFAULT_DIR_MODE, path) end private @@ -41,6 +42,7 @@ module Gitlab def untar_with_options(archive:, dir:, options:) execute(%W(tar -#{options} #{archive} -C #{dir})) + execute(%W(chmod -R #{UNTAR_MASK} #{dir})) end def execute(cmd) |