html-proofer 3.7.6 → 3.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e63d485fbb8116a5955c0e19f8e654477e9c7535
-  data.tar.gz: d6c07f007f28493f5ffd02cf0651efb76f55aa9c
+  metadata.gz: fbaac895cc16c29554e70a2640f654087f7f6bfa
+  data.tar.gz: d0705afd3249cc486bd6de557edb479ea976d15d
 SHA512:
-  metadata.gz: ebd8131b46e01cde307e97e3081aff0665d947e5f11166368a0bbc42b424b665bc8c0ffee772fafb91ed9312d97901cfe4d3b45c0f61a41685f74953359bfb0f
-  data.tar.gz: 9287913635a8cfe96b5c85c8afaf3ee4bbd0984c8b5ed761e18b1d399d38a4fd9d32119193aa4ea3e1cf001dde68556e1bea40ce5c9703761304e9ed9aa76275
+  metadata.gz: b429b5ab045115c0d8a1ac7e527b7240be21650ddf8dcc09f71c3669345a5a0b4e6d43be2174ae8dfc882b3fa33760561e5d8b4e9c93fdab4d8a0d03709c97a8
+  data.tar.gz: 40ccce6b3054b72099e6dfd4db6219dca204dae3fcd8f0301e27870d92392289493aa99f62e9279357464bcd1434c9b18d7797fefcdc4ffeb090e0382ddf5986

data/lib/html-proofer/cache.rb

@@ -128,12 +128,6 @@ module HTMLProofer
       urls_to_check
     end
 
-    # FIXME: there seems to be some discrepenacy where Typhoeus occasionally adds
-    # a trailing slash to URL strings, which causes issues with the cache
-    def slashless_url(url)
-      url.chomp('/')
-    end
-
     # FIXME: it seems that Typhoeus actually acts on escaped URLs,
     # but there's no way to get at that information, and the cache
     # stores unescaped URLs. Because of this, some links, such as
@@ -144,7 +138,7 @@ module HTMLProofer
     end
 
     def clean_url(url)
-      slashless_url(unescape_url(url))
+      unescape_url(url)
     end
 
     def setup_cache!(options)

data/lib/html-proofer/url_validator.rb

@@ -163,7 +163,7 @@ module HTMLProofer
       else
         return if @options[:only_4xx] && !response_code.between?(400, 499)
         # Received a non-successful http response.
-        msg = "External link #{href.chomp('/')} failed: #{response_code} #{response.return_message}"
+        msg = "External link #{href} failed: #{response_code} #{response.return_message}"
         add_external_issue(filenames, msg, response_code)
         @cache.add(href, filenames, response_code, msg)
       end
@@ -186,21 +186,21 @@ module HTMLProofer
 
       return unless body_doc.xpath(xpath).empty?
 
-      msg = "External link #{href.chomp('/')} failed: #{effective_url} exists, but the hash '#{hash}' does not"
+      msg = "External link #{href} failed: #{effective_url} exists, but the hash '#{hash}' does not"
       add_external_issue(filenames, msg, response.code)
       @cache.add(href, filenames, response.code, msg)
       true
     end
 
     def handle_timeout(href, filenames, response_code)
-      msg = "External link #{href.chomp('/')} failed: got a time out (response code #{response_code})"
+      msg = "External link #{href} failed: got a time out (response code #{response_code})"
       @cache.add(href, filenames, 0, msg)
       return if @options[:only_4xx]
       add_external_issue(filenames, msg, response_code)
     end
 
     def handle_failure(href, filenames, response_code, return_message)
-      msg = "External link #{href.chomp('/')} failed: response code #{response_code} means something's wrong.
+      msg = "External link #{href} failed: response code #{response_code} means something's wrong.
              It's possible libcurl couldn't connect to the server or perhaps the request timed out.
              Sometimes, making too many requests at once also breaks things.
              Either way, the return message (if any) from the server is: #{return_message}"

data/lib/html-proofer/version.rb

@@ -1,3 +1,3 @@
 module HTMLProofer
-  VERSION = '3.7.6'.freeze
+  VERSION = '3.8.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 3.7.6
+  version: 3.8.0
 platform: ruby
 authors:
 - Garen Torikian
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-18 00:00:00.000000000 Z
+date: 2018-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mercenary
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '1.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: yell
   requirement: !ruby/object:Gem::Requirement