diff --git a/jekyll.gemspec b/jekyll.gemspec index b506ffa3..734df2e4 100644 --- a/jekyll.gemspec +++ b/jekyll.gemspec @@ -38,4 +38,5 @@ Gem::Specification.new do |s| s.add_runtime_dependency('jekyll-sass-converter', '~> 1.0') s.add_runtime_dependency('jekyll-watch', '~> 1.1') s.add_runtime_dependency("pathutil", "~> 0.9") + s.add_runtime_dependency('addressable', '~> 2.4') end diff --git a/lib/jekyll/filters/url_filters.rb b/lib/jekyll/filters/url_filters.rb index 39f60230..ccc4dcb9 100644 --- a/lib/jekyll/filters/url_filters.rb +++ b/lib/jekyll/filters/url_filters.rb @@ -1,3 +1,5 @@ +require "addressable/uri" + module Jekyll module Filters module URLFilters @@ -10,7 +12,7 @@ module Jekyll return if input.nil? site = @context.registers[:site] return relative_url(input).to_s if site.config["url"].nil? - URI(site.config["url"] + relative_url(input)).to_s + Addressable::URI.parse(site.config["url"] + relative_url(input)).normalize.to_s end # Produces a URL relative to the domain root based on site.baseurl. @@ -22,9 +24,9 @@ module Jekyll return if input.nil? site = @context.registers[:site] return ensure_leading_slash(input.to_s) if site.config["baseurl"].nil? - URI( + Addressable::URI.parse( ensure_leading_slash(site.config["baseurl"]) + ensure_leading_slash(input.to_s) - ).to_s + ).normalize.to_s end private diff --git a/test/test_filters.rb b/test/test_filters.rb index 8ae69039..9a0f87d4 100644 --- a/test/test_filters.rb +++ b/test/test_filters.rb @@ -397,7 +397,7 @@ class TestFilters < JekyllUnitTest should "normalize international URLs" do page_url = "错误.html" - assert_equal "%E9%94%99%E8%AF%AF.html", @filter.relative_url(page_url) + assert_equal "/base/%E9%94%99%E8%AF%AF.html", @filter.relative_url(page_url) end should "be ok with a nil 'baseurl'" do