Add support for Unicode links

* In function 'external?', URI is now escaped before split. * Class Addressable is used for parsing since it is capable of parsing Unicode URIs (IRIs). Another option would be to escape the URI before parsing but URI.escape is deprecated and alternatives (e.g. CGI.escape) work differently. * Resolves #33.
keithmifsud · Dec 5, 2020 · 6438850 · 6438850
1 parent f6fb51a
commit 6438850
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/jekyll-target-blank.rb b/lib/jekyll-target-blank.rb
@@ -193,7 +193,7 @@ def not_mailto_link?(link)
       # link - a url.
       def external?(link)
         if link&.match?(URI.regexp(%w(http https)))
-          URI.parse(link).host != URI.parse(@site_url).host
+          Addressable::URI.parse(link).host != Addressable::URI.parse(@site_url).host
         end
       end
 

diff --git a/lib/jekyll-target-blank/version.rb b/lib/jekyll-target-blank/version.rb
@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module JekyllTargetBlank
-  VERSION = "2.0.0"
+  VERSION = "2.0.1"
 end