diff options
author | Douwe Maan <douwe@gitlab.com> | 2018-01-19 08:44:55 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2018-01-19 08:44:55 +0000 |
commit | 812413863bec3b1f5f3fcf8a8b71906fe5ca2949 (patch) | |
tree | 3fc7001847e8a02a1294010e65f03bc2a4be3636 /lib | |
parent | a1c0964d7170a98214c37e1a45ce8223c83a33f5 (diff) | |
parent | 3b0235318ef1fd39311910da7eb1893a4d3ae312 (diff) | |
download | gitlab-ce-812413863bec3b1f5f3fcf8a8b71906fe5ca2949.tar.gz |
Merge branch '42159-utf8-uploads' into 'master'
Correctly escape UTF-8 path elements for uploads
Closes #42159
See merge request gitlab-org/gitlab-ce!16560
Diffstat (limited to 'lib')
-rw-r--r-- | lib/banzai/filter/relative_link_filter.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/banzai/filter/relative_link_filter.rb b/lib/banzai/filter/relative_link_filter.rb index f6169b2c85d..9bdedeb6615 100644 --- a/lib/banzai/filter/relative_link_filter.rb +++ b/lib/banzai/filter/relative_link_filter.rb @@ -50,7 +50,7 @@ module Banzai end def process_link_to_upload_attr(html_attr) - path_parts = [html_attr.value] + path_parts = [Addressable::URI.unescape(html_attr.value)] if group path_parts.unshift(relative_url_root, 'groups', group.full_path, '-') @@ -58,13 +58,13 @@ module Banzai path_parts.unshift(relative_url_root, project.full_path) end - path = File.join(*path_parts) + path = Addressable::URI.escape(File.join(*path_parts)) html_attr.value = if context[:only_path] path else - URI.join(Gitlab.config.gitlab.base_url, path).to_s + Addressable::URI.join(Gitlab.config.gitlab.base_url, path).to_s end end |