bypass 0.0.2 → 0.0.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDBiMzE5MzA3NjVlMDg2MjE4NDdiMzM1MmIwNjg4N2RhNjcwODRiNQ==
+    MGE5YjdiM2QyNTc1MTM4MzQ3NGVhODRmM2ZiM2ZhMDAzYmZjZTVhZg==
   data.tar.gz: !binary |-
-    OWJlMGQyMjliODRmOGM3YTExZjk5NmIwZThiMmI3ZTgyNDhiYTU2OQ==
+    MGU0ODk2YmVmN2Q3ZjNiNjZiZGI3YmNlYjNlYmVlYzYzZjY0NTA2Yw==
 SHA512:
   metadata.gz: !binary |-
-    OGY3NTZmMzc1ZGRhYzhlZWU2NWJlZDY2OWFmZWQ4YTJmZTkyNjZiZjY1ZWY3
-    OWE3NmFlOWJmYzJhMTg5ZWU4MjVhNTE4ZmE5YWM1OWMzMjU3M2FiNThmMzE2
-    Nzc1OGUzNDc4MWUxODI3YzA4NjVhOTE1NjFhYzM5ZjEwYjJhMWU=
+    NzY2NjUzNDJjYjE3MzBjYzY1ZjQ5NTQ0NGY3ZjFhM2I2YWMyYjYxNWI5MTU3
+    OWNmMzMxMTFhZWI3NzRjNjFiZWUwNjZiNGUyMzc3ZDNiMDA1MzQ5ZmRiMGEy
+    MGE2OWUwY2E5NWJkMjdmYzFjYWI1MTM1OTY4NzQ2YjYwMjA3MDU=
   data.tar.gz: !binary |-
-    M2Y0MTVkZTA1ZmI1YjFkNTNiOGI1NTg5YjM3YWQ2MmZmNDBkNDI1YTIyNjA2
-    MDlmNTFmNGJhOGM3ZjU4YzUyOWIwYTcxOGM1YzM2MWI1ZmI2NWY3MDUyOTky
-    M2M1ZWY4ZmI3OGMyOGY4OWMzN2Q4NzcxY2M1MDUwMjM2ZTA4MWU=
+    YTNmZTcyMzFmMmMxYmNlOTgwZjRiZWI3YjQwM2IwY2ZjMDlmMzU1NjVjMjcz
+    YWU0MGE4OWVhZTY5ODY3Njg3OWIyMDkzMGNmZTg4YmViNWEyYzhhNThiMWQ3
+    NDkyZGE2NDllNzc3OGU2NmMyMjE2ZTI1MTIyZWYzNGU5OTU5Yzg=

data/.travis.yml

@@ -1,11 +1,7 @@
 language: ruby
 rvm:
-  - 1.9.3
+  - 2.0.0
   - 1.9.2
-  - jruby-18mode
   - jruby-19mode
-  - rbx-2.1.1
   - ruby-head
   - jruby-head
-  - 1.8.7
-  - ree

data/lib/bypass/filter.rb

@@ -25,5 +25,11 @@ module Bypass
         yield(match.to_s)
       end
     end
+
+    def parse_uri(uri)
+      Bypass::URI.parse(uri)
+    rescue => ex
+      nil
+    end
   end
 end

data/lib/bypass/html_filter.rb

@@ -6,8 +6,10 @@ module Bypass
     def replace(&block)
       parsed_content.traverse do |node|
         if node.name == "a" && href = node.get_attribute("href")
-          url = yield(Bypass::URI.parse(href))
-          node.set_attribute("href", url.to_s)
+          if parsed_href = parse_uri(href)
+            url = yield(parsed_href)
+            node.set_attribute("href", url.to_s)
+          end
         end
       end
     end

data/lib/bypass/text_filter.rb

@@ -2,8 +2,10 @@ module Bypass
   class TextFilter < Filter
 
     def replace(&block)
-      @content = gsub_urls(content) do |url| 
-        yield(Bypass::URI.parse(url)).to_s
+      @content = gsub_urls(content) do |url|
+        if parsed_url = parse_uri(url)
+          yield(parsed_url).to_s
+        end
       end
     end
 

data/lib/bypass/version.rb

@@ -1,3 +1,3 @@
 module Bypass
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/test/{detour → bypass}/html_filter_test.rb

@@ -16,6 +16,13 @@ class Bypass::HTMLFilterTest < Test::Unit::TestCase
       filter.replace { "foo" }
       assert_equal "<a>Yahoo</a>", filter.content
     end
+
+    should "skip malformed URLs" do
+      text = "<a href=\"http://#\">Yahoo</a>"
+      filter = Bypass::HTMLFilter.new(text)
+      filter.replace { "foo" }
+      assert_equal text, filter.content
+    end
   end
 
   context "#auto_link" do

data/test/{detour → bypass}/text_filter_test.rb

@@ -30,5 +30,12 @@ class Bypass::TextFilterTest < Test::Unit::TestCase
       filter.replace { "foo" }
       assert_equal "foo,", filter.content
     end
+
+    should "skip malformed URLs" do
+      text = "http://#"
+      filter = Bypass::TextFilter.new(text)
+      filter.replace { "foo" }
+      assert_equal text, filter.content
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bypass
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Derrick Reimer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-27 00:00:00.000000000 Z
+date: 2014-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -86,10 +86,10 @@ files:
 - lib/bypass/text_filter.rb
 - lib/bypass/uri.rb
 - lib/bypass/version.rb
-- test/detour/filter_test.rb
-- test/detour/html_filter_test.rb
-- test/detour/text_filter_test.rb
-- test/detour/uri_test.rb
+- test/bypass/filter_test.rb
+- test/bypass/html_filter_test.rb
+- test/bypass/text_filter_test.rb
+- test/bypass/uri_test.rb
 - test/test_helper.rb
 homepage: https://github.com/djreimer/bypass
 licenses: []
@@ -115,8 +115,8 @@ signing_key:
 specification_version: 4
 summary: Mutate URLs and hyperlinks in HTML and plain text documents with ease
 test_files:
-- test/detour/filter_test.rb
-- test/detour/html_filter_test.rb
-- test/detour/text_filter_test.rb
-- test/detour/uri_test.rb
+- test/bypass/filter_test.rb
+- test/bypass/html_filter_test.rb
+- test/bypass/text_filter_test.rb
+- test/bypass/uri_test.rb
 - test/test_helper.rb