html-proofer 5.0.3 → 5.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 337d1725f7139e6efda6d350b83b15ecfd51eb67ee44e068f1b29859104320dd
-  data.tar.gz: 3ad1dcdb66a4789853e92303a1cd6e9872d95db137ab589df75f5f6af00d7ed0
+  metadata.gz: 5480fbee53b67ca43a6a5896919e8d6ab82bca8e31b20e0734b60c78a71747c6
+  data.tar.gz: 12d2e42cefdab3b45b8048e7293e2bb0ccb2235fb6ac2b1722b662f1129a7202
 SHA512:
-  metadata.gz: 656b20f4feeffc2dca48e38e42d510c8b7f5d4e7f563167429e0c687c0b1a938008dfe8bb7e21591ed8a5c76fd7ea8c20d63b78f2c71fcf54a3a9bcf55970a7e
-  data.tar.gz: 490f87dcd3a849a55c2051e56e93e8b65e47e18ffea0fd82061b8418736709cff5ec5230fd84d937e7b748ccec19c644650ef970dda49a0ce100f57914be6418
+  metadata.gz: 6789de48b6d391cb36fd1c7b05bba18960e1782e4ef20b9cf15c076efdb147231a7b56c7c79eaad0c4c267a58fbc3355cb8a063334a7224a8901ac8a594908d8
+  data.tar.gz: 17fb7a62b96b1eb3b47dd297f8aedab8e02ffb2eacf3880d1b4ea7b0061ab4ba13dcc20993e76ba36d766d29f6ef8726823e5f73524b01eea5de96cba2455e96

data/lib/html_proofer/attribute/url.rb

@@ -141,12 +141,16 @@ module HTMLProofer
           # either overwrite with root_dir; or, if source is directory, use that; or, just get the current file's dirname
           @runner.options[:root_dir] || (File.directory?(@runner.current_source) ? @runner.current_source : File.dirname(@runner.current_source))
         # relative links, path is a file
-        elsif File.exist?(File.expand_path(path,
-          @runner.current_source)) || File.exist?(File.expand_path(path_dot_ext, @runner.current_source))
+        elsif File.exist?(File.expand_path(
+          path,
+          @runner.current_source,
+        )) || File.exist?(File.expand_path(path_dot_ext, @runner.current_source))
           File.dirname(@runner.current_filename)
         # relative links in nested dir, path is a file
-        elsif File.exist?(File.join(File.dirname(@runner.current_filename),
-          path)) || File.exist?(File.join(File.dirname(@runner.current_filename), path_dot_ext))
+        elsif File.exist?(File.join(
+          File.dirname(@runner.current_filename),
+          path,
+        )) || File.exist?(File.join(File.dirname(@runner.current_filename), path_dot_ext))
           File.dirname(@runner.current_filename)
         # relative link, path is a directory
         else
@@ -213,7 +217,7 @@ module HTMLProofer
         url.start_with?("?")
       end
 
-      def sans_hash
+      def without_hash
         @url.to_s.sub(/##{hash}/, "")
       end
 

data/lib/html_proofer/cache.rb

@@ -93,8 +93,10 @@ module HTMLProofer
       # if there are no urls, bail
       return {} if urls_detected.empty?
 
-      urls_detected = urls_detected.transform_keys do |url|
-        cleaned_url(url)
+      if type == :external
+        urls_detected = urls_detected.transform_keys do |url|
+          cleaned_url(url)
+        end
       end
 
       urls_to_check = detect_url_changes(urls_detected, type)

data/lib/html_proofer/check/favicon.rb

@@ -17,13 +17,19 @@ module HTMLProofer
 
         if found
           if @favicon.url.protocol_relative?
-            add_failure("favicon link #{@favicon.url} is a protocol-relative URL, use explicit https:// instead",
-              line: @favicon.line, content: @favicon.content)
+            add_failure(
+              "favicon link #{@favicon.url} is a protocol-relative URL, use explicit https:// instead",
+              line: @favicon.line,
+              content: @favicon.content,
+            )
           elsif @favicon.url.remote?
             add_to_external_urls(@favicon.url, @favicon.line)
           elsif !@favicon.url.exists?
-            add_failure("internal favicon #{@favicon.url.raw_attribute} does not exist", line: @favicon.line,
-              content: @favicon.content)
+            add_failure(
+              "internal favicon #{@favicon.url.raw_attribute} does not exist",
+              line: @favicon.line,
+              content: @favicon.content,
+            )
           end
         else
           add_failure("no favicon provided")

data/lib/html_proofer/check/images.rb

@@ -12,27 +12,39 @@ module HTMLProofer
           next if @img.ignore?
 
           # screenshot filenames should return because of terrible names
-          add_failure("image has a terrible filename (#{@img.url.raw_attribute})", line: @img.line,
-            content: @img.content) if terrible_filename?
+          add_failure(
+            "image has a terrible filename (#{@img.url.raw_attribute})",
+            line: @img.line,
+            content: @img.content,
+          ) if terrible_filename?
 
           # does the image exist?
           if missing_src?
             add_failure("image has no src or srcset attribute", line: @img.line, content: @img.content)
           elsif @img.url.protocol_relative?
-            add_failure("image link #{@img.url} is a protocol-relative URL, use explicit https:// instead",
-              line: @img.line, content: @img.content)
+            add_failure(
+              "image link #{@img.url} is a protocol-relative URL, use explicit https:// instead",
+              line: @img.line,
+              content: @img.content,
+            )
           elsif @img.url.remote?
             add_to_external_urls(@img.url, @img.line)
           elsif !@img.url.exists? && !@img.multiple_srcsets? && !@img.multiple_sizes?
-            add_failure("internal image #{@img.url.raw_attribute} does not exist", line: @img.line,
-              content: @img.content)
+            add_failure(
+              "internal image #{@img.url.raw_attribute} does not exist",
+              line: @img.line,
+              content: @img.content,
+            )
           elsif @img.multiple_srcsets? || @img.multiple_sizes?
             @img.srcsets_wo_sizes.each do |srcset|
               srcset_url = HTMLProofer::Attribute::Url.new(@runner, srcset, base_url: @img.base_url, extract_size: true)
 
               if srcset_url.protocol_relative?
-                add_failure("image link #{srcset_url.url} is a protocol-relative URL, use explicit https:// instead",
-                  line: @img.line, content: @img.content)
+                add_failure(
+                  "image link #{srcset_url.url} is a protocol-relative URL, use explicit https:// instead",
+                  line: @img.line,
+                  content: @img.content,
+                )
               elsif srcset_url.remote?
                 add_to_external_urls(srcset_url.url, @img.line)
               elsif !srcset_url.exists?
@@ -44,16 +56,25 @@ module HTMLProofer
           # if this is an img element, check that the alt attribute is present
           if @img.img_tag? && !ignore_element?
             if missing_alt_tag? && !ignore_missing_alt?
-              add_failure("image #{@img.url.raw_attribute} does not have an alt attribute", line: @img.line,
-                content: @img.content)
+              add_failure(
+                "image #{@img.url.raw_attribute} does not have an alt attribute",
+                line: @img.line,
+                content: @img.content,
+              )
             elsif (empty_alt_tag? || alt_all_spaces?) && !ignore_empty_alt?
-              add_failure("image #{@img.url.raw_attribute} has an alt attribute, but no content", line: @img.line,
-                content: @img.content)
+              add_failure(
+                "image #{@img.url.raw_attribute} has an alt attribute, but no content",
+                line: @img.line,
+                content: @img.content,
+              )
             end
           end
 
-          add_failure("image #{@img.url.raw_attribute} uses the http scheme", line: @img.line,
-            content: @img.content) if @runner.enforce_https? && @img.url.http?
+          add_failure(
+            "image #{@img.url.raw_attribute} uses the http scheme",
+            line: @img.line,
+            content: @img.content,
+          ) if @runner.enforce_https? && @img.url.http?
         end
 
         external_urls

data/lib/html_proofer/check/links.rb

@@ -29,8 +29,11 @@ module HTMLProofer
           end
 
           if @link.url.protocol_relative?
-            add_failure("#{@link.url} is a protocol-relative URL, use explicit https:// instead",
-              line: @link.line, content: @link.content)
+            add_failure(
+              "#{@link.url} is a protocol-relative URL, use explicit https:// instead",
+              line: @link.line,
+              content: @link.content,
+            )
             next
           end
 
@@ -55,8 +58,11 @@ module HTMLProofer
           elsif @link.url.internal?
             # does the local directory have a trailing slash?
             if @link.url.unslashed_directory?(@link.url.absolute_path)
-              add_failure("internally linking to a directory #{@link.url.raw_attribute} without trailing slash",
-                line: @link.line, content: @link.content)
+              add_failure(
+                "internally linking to a directory #{@link.url.raw_attribute} without trailing slash",
+                line: @link.line,
+                content: @link.content,
+              )
               next
             end
 
@@ -88,17 +94,26 @@ module HTMLProofer
 
       def handle_mailto
         if @link.url.path.empty?
-          add_failure("#{@link.url.raw_attribute} contains no email address", line: @link.line,
-            content: @link.content) unless ignore_empty_mailto?
+          add_failure(
+            "#{@link.url.raw_attribute} contains no email address",
+            line: @link.line,
+            content: @link.content,
+          ) unless ignore_empty_mailto?
         elsif !/#{URI::MailTo::EMAIL_REGEXP}/o.match?(@link.url.path)
-          add_failure("#{@link.url.raw_attribute} contains an invalid email address", line: @link.line,
-            content: @link.content)
+          add_failure(
+            "#{@link.url.raw_attribute} contains an invalid email address",
+            line: @link.line,
+            content: @link.content,
+          )
         end
       end
 
       def handle_tel
-        add_failure("#{@link.url.raw_attribute} contains no phone number", line: @link.line,
-          content: @link.content) if @link.url.path.empty?
+        add_failure(
+          "#{@link.url.raw_attribute} contains no phone number",
+          line: @link.line,
+          content: @link.content,
+        ) if @link.url.path.empty?
       end
 
       def ignore_empty_mailto?
@@ -113,13 +128,19 @@ module HTMLProofer
         return unless SRI_REL_TYPES.include?(@link.node["rel"])
 
         if blank?(@link.node["integrity"]) && blank?(@link.node["crossorigin"])
-          add_failure("SRI and CORS not provided in: #{@link.url.raw_attribute}", line: @link.line,
-            content: @link.content)
+          add_failure(
+            "SRI and CORS not provided in: #{@link.url.raw_attribute}",
+            line: @link.line,
+            content: @link.content,
+          )
         elsif blank?(@link.node["integrity"])
           add_failure("Integrity is missing in: #{@link.url.raw_attribute}", line: @link.line, content: @link.content)
         elsif blank?(@link.node["crossorigin"])
-          add_failure("CORS not provided for external resource in: #{@link.link.url.raw_attribute}", line: @link.line,
-            content: @link.content)
+          add_failure(
+            "CORS not provided for external resource in: #{@link.link.url.raw_attribute}",
+            line: @link.line,
+            content: @link.content,
+          )
         end
       end
 

data/lib/html_proofer/check/open_graph.rb

@@ -17,13 +17,19 @@ module HTMLProofer
           elsif !@open_graph.url.valid?
             add_failure("#{@open_graph.src} is an invalid URL", line: @open_graph.line)
           elsif @open_graph.url.protocol_relative?
-            add_failure("open graph link #{@open_graph.url} is a protocol-relative URL, use explicit https:// instead",
-              line: @open_graph.line, content: @open_graph.content)
+            add_failure(
+              "open graph link #{@open_graph.url} is a protocol-relative URL, use explicit https:// instead",
+              line: @open_graph.line,
+              content: @open_graph.content,
+            )
           elsif @open_graph.url.remote?
             add_to_external_urls(@open_graph.url, @open_graph.line)
           else
-            add_failure("internal open graph #{@open_graph.url.raw_attribute} does not exist", line: @open_graph.line,
-              content: @open_graph.content) unless @open_graph.url.exists?
+            add_failure(
+              "internal open graph #{@open_graph.url.raw_attribute} does not exist",
+              line: @open_graph.line,
+              content: @open_graph.content,
+            ) unless @open_graph.url.exists?
           end
         end
 

data/lib/html_proofer/check/scripts.rb

@@ -14,14 +14,20 @@ module HTMLProofer
           if missing_src?
             add_failure("script is empty and has no src attribute", line: @script.line, content: @script.content)
           elsif @script.url.protocol_relative?
-            add_failure("script link #{@script.url} is a protocol-relative URL, use explicit https:// instead",
-              line: @script.line, content: @script.content)
+            add_failure(
+              "script link #{@script.url} is a protocol-relative URL, use explicit https:// instead",
+              line: @script.line,
+              content: @script.content,
+            )
           elsif @script.url.remote?
             add_to_external_urls(@script.url, @script.line)
             check_sri if @runner.check_sri?
           elsif !@script.url.exists?
-            add_failure("internal script reference #{@script.src} does not exist", line: @script.line,
-              content: @script.content)
+            add_failure(
+              "internal script reference #{@script.src} does not exist",
+              line: @script.line,
+              content: @script.content,
+            )
           end
         end
 
@@ -34,14 +40,23 @@ module HTMLProofer
 
       def check_sri
         if blank?(@script.node["integrity"]) && blank?(@script.node["crossorigin"])
-          add_failure("SRI and CORS not provided in: #{@script.url.raw_attribute}", line: @script.line,
-            content: @script.content)
+          add_failure(
+            "SRI and CORS not provided in: #{@script.url.raw_attribute}",
+            line: @script.line,
+            content: @script.content,
+          )
         elsif blank?(@script.node["integrity"])
-          add_failure("Integrity is missing in: #{@script.url.raw_attribute}", line: @script.line,
-            content: @script.content)
+          add_failure(
+            "Integrity is missing in: #{@script.url.raw_attribute}",
+            line: @script.line,
+            content: @script.content,
+          )
         elsif blank?(@script.node["crossorigin"])
-          add_failure("CORS not provided for external resource in: #{@script.url.raw_attribute}", line: @script.line,
-            content: @script.content)
+          add_failure(
+            "CORS not provided for external resource in: #{@script.url.raw_attribute}",
+            line: @script.line,
+            content: @script.content,
+          )
         end
       end
     end

data/lib/html_proofer/check.rb

@@ -25,8 +25,14 @@ module HTMLProofer
     end
 
     def add_failure(description, line: nil, status: nil, content: nil)
-      @failures << Failure.new(@runner.current_filename, short_name, description, line: line, status: status,
-        content: content)
+      @failures << Failure.new(
+        @runner.current_filename,
+        short_name,
+        description,
+        line: line,
+        status: status,
+        content: content,
+      )
     end
 
     def short_name

data/lib/html_proofer/configuration.rb

@@ -280,17 +280,25 @@ module HTMLProofer
     module ConfigurationHelp
       TEXT = {
         as_links: ["Assumes that `PATH` is a comma-separated array of links to check."],
-        assume_extension: ["Automatically add specified extension to files for internal links, ",
-                           "to allow extensionless URLs (as supported by most servers) (default: `.html`).",],
+        assume_extension: [
+          "Automatically add specified extension to files for internal links, ",
+          "to allow extensionless URLs (as supported by most servers) (default: `.html`).",
+        ],
         directory_index_file: ["Sets the file to look for when a link refers to a directory. (default: `index.html`)."],
-        extensions: ["A comma-separated list of Strings indicating the file extensions you",
-                     "would like to check (default: `.html`)",],
+        extensions: [
+          "A comma-separated list of Strings indicating the file extensions you",
+          "would like to check (default: `.html`)",
+        ],
 
         allow_hash_href: ['"If `true`, assumes `href="#"` anchors are valid (default: `true`)"'],
-        allow_missing_href: ["If `true`, does not flag `a` tags missing `href`. In HTML5, this is technically ",
-                             "allowed, but could also be human error. (default: `false`)",],
-        checks: ["A comma-separated list of Strings indicating which checks you",
-                 "want to run (default: `[\"Links\", \"Images\", \"Scripts\"]",],
+        allow_missing_href: [
+          "If `true`, does not flag `a` tags missing `href`. In HTML5, this is technically ",
+          "allowed, but could also be human error. (default: `false`)",
+        ],
+        checks: [
+          "A comma-separated list of Strings indicating which checks you",
+          "want to run (default: `[\"Links\", \"Images\", \"Scripts\"]",
+        ],
         check_external_hash: ["Checks whether external hashes exist (even if the webpage exists) (default: `true`)."],
         check_internal_hash: ["Checks whether internal hashes exist (even if the webpage exists) (default: `true`)."],
         check_sri: ["Check that `<link>` and `<script>` external resources use SRI (default: `false`)."],
@@ -298,31 +306,43 @@ module HTMLProofer
         enforce_https: ["Fails a link if it's not marked as `https` (default: `true`)."],
         root_dir: ["The absolute path to the directory serving your html-files."],
 
-        ignore_empty_alt: ["If `true`, ignores images with empty/missing ",
-                           "alt tags (in other words, `<img alt>` and `<img alt=\"\">`",
-                           "are valid; set this to `false` to flag those) (default: `true`).",],
-        ignore_empty_mailto: ["If `true`, allows `mailto:` `href`s which don't",
-                              "contain an email address (default: `false`)'.",],
+        ignore_empty_alt: [
+          "If `true`, ignores images with empty/missing ",
+          "alt tags (in other words, `<img alt>` and `<img alt=\"\">`",
+          "are valid; set this to `false` to flag those) (default: `true`).",
+        ],
+        ignore_empty_mailto: [
+          "If `true`, allows `mailto:` `href`s which don't",
+          "contain an email address (default: `false`)'.",
+        ],
         ignore_missing_alt: ["If `true`, ignores images with missing alt tags (default: `false`)."],
         ignore_status_codes: ["A comma-separated list of numbers representing status codes to ignore."],
         ignore_files: ["A comma-separated list of Strings or RegExps containing file paths that are safe to ignore"],
-        ignore_urls: ["A comma-separated list of Strings or RegExps containing URLs that are",
-                      "safe to ignore. This affects all HTML attributes, such as `alt` tags on images.",],
+        ignore_urls: [
+          "A comma-separated list of Strings or RegExps containing URLs that are",
+          "safe to ignore. This affects all HTML attributes, such as `alt` tags on images.",
+        ],
         only_status_codes: ["A comma-separated list of numbers representing the only status codes to report on."],
         only_4xx: ["Only reports errors for links that fall within the 4xx status code range."],
 
-        swap_attributes: ["JSON-formatted config that maps element names to the",
-                          "preferred attribute to check (default: `{}`).",],
-        swap_urls: ["A comma-separated list containing key-value pairs of `RegExp => String`.",
-                    "It transforms URLs that match `RegExp` into `String` via `gsub`.",
-                    "The escape sequences `\\:` should be used to produce literal `:`s.",],
+        swap_attributes: [
+          "JSON-formatted config that maps element names to the",
+          "preferred attribute to check (default: `{}`).",
+        ],
+        swap_urls: [
+          "A comma-separated list containing key-value pairs of `RegExp => String`.",
+          "It transforms URLs that match `RegExp` into `String` via `gsub`.",
+          "The escape sequences `\\:` should be used to produce literal `:`s.",
+        ],
 
         typhoeus: ["JSON-formatted string of Typhoeus config; if set, overrides the html-proofer defaults."],
         hydra: ["JSON-formatted string of Hydra config; if set, overrides the html-proofer defaults."],
         cache: ["JSON-formatted string of cache config; if set, overrides the html-proofer defaults."],
 
-        log_level: ["Sets the logging level. One of `:debug`, `:info`, ",
-                    "`:warn`, `:error`, or `:fatal`. (default: `:info`)",],
+        log_level: [
+          "Sets the logging level. One of `:debug`, `:info`, ",
+          "`:warn`, `:error`, or `:fatal`. (default: `:info`)",
+        ],
 
         version: ["Prints the version of html-proofer."],
       }.freeze

data/lib/html_proofer/log.rb

@@ -11,9 +11,11 @@ module HTMLProofer
     STDERR_LEVELS = [:error, :fatal].freeze
 
     def initialize(log_level)
-      @logger = Yell.new(format: false, \
+      @logger = Yell.new(
+        format: false, \
         name: "HTMLProofer", \
-        level: "gte.#{log_level}") do |l|
+        level: "gte.#{log_level}",
+      ) do |l|
         l.adapter(:stdout, level: "lte.warn")
         l.adapter(:stderr, level: "gte.error")
       end

data/lib/html_proofer/runner.rb

@@ -42,8 +42,10 @@ module HTMLProofer
         @logger.log(:info, "Running #{check_text} (#{format_checks_list(checks)}) on #{@source} ...\n\n")
         check_list_of_links unless @options[:disable_external]
       else
-        @logger.log(:info,
-          "Running #{check_text} (#{format_checks_list(checks)}) in #{@source} on *#{@options[:extensions].join(", ")} files ...\n\n")
+        @logger.log(
+          :info,
+          "Running #{check_text} (#{format_checks_list(checks)}) in #{@source} on *#{@options[:extensions].join(", ")} files ...\n\n",
+        )
 
         check_files
         @logger.log(:info, "Ran on #{pluralize(files.length, "file", "files")}!\n\n")

data/lib/html_proofer/url_validator/external.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
 require "typhoeus"
-require "uri"
+require "open-uri"
+# require "uri"
+require "pdf-reader"
 
 module HTMLProofer
   class UrlValidator
@@ -88,8 +90,12 @@ module HTMLProofer
         return if @runner.options[:ignore_status_codes].include?(response_code)
 
         if response_code.between?(200, 299)
-          @cache.add_external(href, filenames, response_code, "OK", true) unless check_hash_in_2xx_response(href, url,
-            response, filenames)
+          @cache.add_external(href, filenames, response_code, "OK", true) unless check_hash_in_2xx_response(
+            href,
+            url,
+            response,
+            filenames,
+          )
         elsif response.timed_out?
           handle_timeout(href, filenames, response_code)
         elsif response_code.zero?
@@ -115,6 +121,28 @@ module HTMLProofer
         return false unless url.hash?
 
         hash = url.hash
+        headers = response.options.fetch(:headers, {})
+        content_type = headers.find { |k, _| k.casecmp("content-type").zero? }
+
+        # attempt to verify PDF hash ref; see #787 for more details
+        # FIXME: this is re-reading the PDF response
+        if content_type && content_type[1].include?("pdf")
+          io = URI.parse(url.to_s).open
+          reader = PDF::Reader.new(io)
+
+          pages = reader.pages
+          if hash =~ /\Apage=(\d+)\z/
+            page = Regexp.last_match[1].to_i
+
+            unless pages[page - 1]
+              msg = "External link #{href} failed: #{url.without_hash} exists, but the hash '#{hash}' does not"
+              add_failure(filenames, msg, response.code)
+              @cache.add_external(href, filenames, response.code, msg, false)
+            end
+
+            return true
+          end
+        end
 
         body_doc = create_nokogiri(response.body)
 
@@ -130,7 +158,7 @@ module HTMLProofer
 
         return unless body_doc.xpath(xpath.join("|")).empty?
 
-        msg = "External link #{href} failed: #{url.sans_hash} exists, but the hash '#{hash}' does not"
+        msg = "External link #{href} failed: #{url.without_hash} exists, but the hash '#{hash}' does not"
         add_failure(filenames, msg, response.code)
         @cache.add_external(href, filenames, response.code, msg, false)
         true

data/lib/html_proofer/url_validator/internal.rb

@@ -36,8 +36,14 @@ module HTMLProofer
 
             target_file_path = url.absolute_path
             unless file_exists?(target_file_path)
-              @failed_checks << Failure.new(@runner.current_filename, "Links > Internal",
-                "internally linking to #{url}, which does not exist", line: metadata[:line], status: nil, content: nil)
+              @failed_checks << Failure.new(
+                @runner.current_filename,
+                "Links > Internal",
+                "internally linking to #{url}, which does not exist",
+                line: metadata[:line],
+                status: nil,
+                content: nil,
+              )
               to_add << [url, metadata, false]
               next
             end
@@ -55,8 +61,14 @@ module HTMLProofer
               next
             end
             unless hash_exists
-              @failed_checks << Failure.new(@runner.current_filename, "Links > Internal",
-                "internally linking to #{url}; the file exists, but the hash '#{url.hash}' does not", line: metadata[:line], status: nil, content: nil)
+              @failed_checks << Failure.new(
+                @runner.current_filename,
+                "Links > Internal",
+                "internally linking to #{url}; the file exists, but the hash '#{url.hash}' does not",
+                line: metadata[:line],
+                status: nil,
+                content: nil,
+              )
               to_add << [url, metadata, false]
               next
             end
@@ -75,8 +87,14 @@ module HTMLProofer
             exists = hash_exists_in_html?(href_hash, html)
             url_metadata.each do |(url, metadata)|
               unless exists
-                @failed_checks << Failure.new(metadata[:filename], "Links > Internal",
-                  "internally linking to #{url}; the file exists, but the hash '#{href_hash}' does not", line: metadata[:line], status: nil, content: nil)
+                @failed_checks << Failure.new(
+                  metadata[:filename],
+                  "Links > Internal",
+                  "internally linking to #{url}; the file exists, but the hash '#{href_hash}' does not",
+                  line: metadata[:line],
+                  status: nil,
+                  content: nil,
+                )
               end
               to_add << [url, metadata, exists]
             end

data/lib/html_proofer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTMLProofer
-  VERSION = "5.0.3"
+  VERSION = "5.0.5"
 end

data/lib/html_proofer/xpath_functions.rb

@@ -4,7 +4,7 @@ module HTMLProofer
   # https://stackoverflow.com/a/8812293
   class XpathFunctions
     def case_sensitive_equals(node_set, str_to_match)
-      node_set.find_all { |node| node.to_s.== str_to_match.to_s }
+      node_set.find_all { |node| node.to_s == str_to_match.to_s }
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 5.0.3
+  version: 5.0.5
 platform: ruby
 authors:
 - Garen Torikian
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-12-05 00:00:00.000000000 Z
+date: 2023-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -52,6 +52,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.13'
+- !ruby/object:Gem::Dependency
+  name: pdf-reader
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.11'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.11'
 - !ruby/object:Gem::Dependency
   name: rainbow
   requirement: !ruby/object:Gem::Requirement
@@ -108,62 +122,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.5'
-- !ruby/object:Gem::Dependency
-  name: awesome_print
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: debug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: redcarpet
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -178,48 +136,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop-rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop-standard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement