Do not blow up on Addressable::IDNA::PunycodeBigOutput

Related to #112.
twingly · Mar 23, 2017 · 3f2a1be · 3f2a1be
1 parent 1a771ea
commit 3f2a1be
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/lib/twingly/url.rb b/lib/twingly/url.rb
@@ -1,3 +1,4 @@
+require "addressable/idna"
 require "addressable/uri"
 require "public_suffix"
 
@@ -14,6 +15,7 @@ class URL
     CUSTOM_PSL = PublicSuffixList.with_punycoded_names
     ENDS_WITH_SLASH = /\/+$/
     ERRORS_TO_EXTEND = [
+      Addressable::IDNA::PunycodeBigOutput,
       Addressable::URI::InvalidURIError,
       PublicSuffix::DomainInvalid,
     ]

diff --git a/spec/lib/twingly/url_spec.rb b/spec/lib/twingly/url_spec.rb
@@ -43,6 +43,9 @@ def invalid_urls
     # https://github.com/sporkmonger/addressable/issues/224
     "http://some_site.net%C2",
     "http://+%D5d.some_site.net",
+
+    # Triggers Addressable::IDNA::PunycodeBigOutput
+    "http://40world-many.ru&amp;passwd=pUXFGc0LS5&amp;subject=%D0%B1%D0%B0%D0%BB%D0%B0%D0%BD%D1%81%D0%B8%D1%80%D0%BE%D0%B2%D0%BA%D0%B0+%D0%BA%D0%B0%D1%80%D0%B4%D0%B0%D0%BD%D0%BD%D0%BE%D0%B3%D0%BE+%D0%B2%D0%B0%D0%BB%D0%B0&amp;commit=Predict&amp;complex=true&amp;complex=false&amp;membrane=false&amp;coil=false&amp;msa_control=all&amp;secStructPred=true&amp;secStructPred=false&amp;falseRate=5&amp;output=opnone&amp;modeller=&amp;seqalign=yes&amp;database=PfamA&amp;eval=0.01&amp;iterations=5&amp;domssea=yes&amp;secpro=yes&amp;pp=yes",
   ]
 end