Merge pull request #496 from erated/fix_parsing_empty_filename_conten…

…t_disposition

support parsing of content disposition header with empty filename

lib/mechanize/parser.rb

@@ -118,13 +118,13 @@ def extract_filename full_path = @full_path
     end
 
     # Set the filename
-    if disposition = @response['content-disposition'] then
+    if (disposition = @response['content-disposition'])
       content_disposition =
         Mechanize::HTTP::ContentDispositionParser.parse disposition
 
-      if content_disposition && content_disposition.filename && content_disposition.filename != '' then
+      if content_disposition && content_disposition.filename && content_disposition.filename != ''
         filename = content_disposition.filename
-        filename = filename.split(/[\\\/]/).last
+        filename = filename.rpartition(/[\\\/]/).last
         handled = true
       end
     end

test/test_mechanize_parser.rb

@@ -162,6 +162,16 @@ def test_extract_filename_content_disposition_windows_special
     end
   end
 
+  def test_extract_filename_content_disposition_empty
+    @parser.uri = URI 'http://example'
+
+    @parser.response = {
+      'content-disposition' => 'inline; filename="/"'
+    }
+
+    assert_equal '', @parser.extract_filename
+  end
+
   def test_extract_filename_host
     @parser.response = {}
     @parser.uri = URI 'http://example'