Catch all URI errors in ExternalLinkFilter

changelogs/unreleased/dm-catch-uri-errors.yml

@@ -0,0 +1,4 @@
+---
+title: Catch all URI errors in ExternalLinkFilter
+merge_request:
+author:

lib/banzai/filter/external_link_filter.rb

@@ -24,7 +24,7 @@ module Banzai
 
       def uri(href)
         URI.parse(href)
-      rescue URI::InvalidURIError
+      rescue URI::Error
         nil
       end
 

spec/lib/banzai/filter/external_link_filter_spec.rb

@@ -55,6 +55,13 @@ describe Banzai::Filter::ExternalLinkFilter, lib: true do
 
       expect(doc.to_html).to eq(expected)
     end
+
+    it 'skips improperly formatted mailtos' do
+      doc = filter %q(<p><a href="mailto://jblogs@example.com">Email</a></p>)
+      expected = %q(<p><a href="mailto://jblogs@example.com">Email</a></p>)
+
+      expect(doc.to_html).to eq(expected)
+    end
   end
 
   context 'for links with a username' do