Skip to content

Make sure there's only one slash as path separator

Lin Jen-Shin requested to merge ee-53180-append-path into master

What does this MR do?

Make sure there's only one slash as path separator

In Ruby 2.4, URI.join("http://test//", "a").to_s will remove the double slash, however it's not the case in Ruby 2.5. Using chomp should work better for the intention, as we're not trying to allow things like ../ or / paths resolution.

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/53180

Edited by Stan Hu

Merge request reports

Loading