html-proofer 5.0.0 → 5.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f2c984bfe049fba10c746d289d8d0256f219d6ae8f5af152f6f8af68e639301b
-  data.tar.gz: c407f3995e873bbbc42d90de1f082827a40caa90cb12e5cdba7fd514500d1124
+  metadata.gz: d9ed4a509a2d120a3aff5ad4809f6b91feff3842dceb371015aca5fd08e69bb2
+  data.tar.gz: 0db88e877a8fc19022d4fd1f382e4d38bfeead1540c42817e33799ccf2527bc7
 SHA512:
-  metadata.gz: 732365ae4207754f70b27843a52d4f8186673fe1282ddaae03244ebff07517816371e970b75e465539bac51e931d51d4b1856866fcd6a3a575ae6e980d7c6465
-  data.tar.gz: ab01a8425d206863584e6c84f395ab85a22eff9d2e1df0daa4e0f72771fc9bde95a8992da17d2ef8195439ad1de2412135758989bb9ba29766b1978d95e0fad4
+  metadata.gz: 6c888e7975c91eb41124057eba57fb3bdd9f8d82f38be6958c82649458526d7a41cb9aac8d4221678c273db419708cb2abfce2ce0be7396acd2b97295e938844
+  data.tar.gz: 61acc2e9c73ba8ca51abca84e21524e145a3f98f8d0d8db4c9683d13407610ea13dfe04342cc861ca0109cd8f9ac1fa24300bb0df684a198787ee5a2d40c8cde

data/lib/html_proofer/attribute/url.rb

@@ -3,13 +3,16 @@
 module HTMLProofer
   class Attribute
     class Url < HTMLProofer::Attribute
-      attr_reader :url, :size
+      attr_reader :url, :size, :source, :filename
 
       REMOTE_SCHEMES = ["http", "https"].freeze
 
-      def initialize(runner, link_attribute, base_url: nil, extract_size: false)
+      def initialize(runner, link_attribute, base_url: nil, source: nil, filename: nil, extract_size: false)
         super
 
+        @source = source
+        @filename = filename
+
         if @raw_attribute.nil?
           @url = nil
         else
@@ -49,7 +52,8 @@ module HTMLProofer
 
       def ignore?
         return true if /^javascript:/.match?(@url)
-        return true if ignores_pattern?(@runner.options[:ignore_urls])
+
+        true if ignores_pattern?(@runner.options[:ignore_urls])
       end
 
       def valid?
@@ -115,9 +119,29 @@ module HTMLProofer
       def exists?
         return true if base64?
 
-        return @runner.checked_paths[absolute_path] if @runner.checked_paths.key?(absolute_path)
+        !resolved_path.nil?
+      end
+
+      def resolved_path
+        path_to_resolve = absolute_path
 
-        @runner.checked_paths[absolute_path] = File.exist?(absolute_path)
+        return @runner.resolved_paths[path_to_resolve] if @runner.resolved_paths.key?(path_to_resolve)
+
+        # extensionless URLs
+        path_with_extension = "#{path_to_resolve}#{@runner.options[:assume_extension]}"
+        resolved = if @runner.options[:assume_extension] && File.file?(path_with_extension)
+          path_with_extension # existence checked implicitly by File.file?
+        # implicit index support
+        elsif File.directory?(path_to_resolve) && !unslashed_directory?(path_to_resolve)
+          path_with_index = File.join(path_to_resolve, @runner.options[:directory_index_file])
+          path_with_index if File.file?(path_with_index)
+        # explicit file or directory
+        elsif File.exist?(path_to_resolve)
+          path_to_resolve
+        end
+        @runner.resolved_paths[path_to_resolve] = resolved
+
+        resolved
       end
 
       def base64?
@@ -125,43 +149,23 @@ module HTMLProofer
       end
 
       def absolute_path
-        path = file_path || @runner.current_filename
+        path = full_path || @filename
 
         File.expand_path(path, Dir.pwd)
       end
 
-      def file_path
+      def full_path
         return if path.nil? || path.empty?
 
-        path_dot_ext = ""
-
-        path_dot_ext = path + @runner.options[:assume_extension] unless blank?(@runner.options[:assume_extension])
-
         base = if absolute_path?(path) # path relative to root
-          # 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))
-          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))
-          File.dirname(@runner.current_filename)
-        # relative link, path is a directory
+          # either overwrite with root_dir; or, if source is directory, use that; or, just get the source file's dirname
+          @runner.options[:root_dir] || (File.directory?(@source) ? @source : File.dirname(@source))
         else
-          @runner.current_filename
-        end
-
-        file = File.join(base, path)
-
-        if @runner.options[:assume_extension] && File.file?("#{file}#{@runner.options[:assume_extension]}")
-          file = "#{file}#{@runner.options[:assume_extension]}"
-        elsif File.directory?(file) && !unslashed_directory?(file) # implicit index support
-          file = File.join(file, @runner.options[:directory_index_file])
+          # path relative to the file where the link is defined
+          File.dirname(@filename)
         end
 
-        file
+        File.join(base, path)
       end
 
       def unslashed_directory?(file)
@@ -213,15 +217,28 @@ module HTMLProofer
         url.start_with?("?")
       end
 
-      def sans_hash
+      def without_hash
         @url.to_s.sub(/##{hash}/, "")
       end
 
-      # catch any obvious issues, like strings in port numbers
+      # catch any obvious issues
       private def clean_url!
-        return if @url =~ /^([!#{Regexp.last_match(0)}-;=?-\[\]_a-z~]|%[0-9a-fA-F]{2})+$/
-
-        @url = Addressable::URI.parse(@url).normalize.to_s
+        parsed_url = Addressable::URI.parse(@url)
+        url = if parsed_url.scheme.nil?
+          parsed_url
+        else
+          parsed_url.normalize
+        end.to_s
+
+        # normalize strips this off, which causes issues with cache
+        @url = if @url.end_with?("/") && !url.end_with?("/")
+          "#{url}/"
+        elsif !@url.end_with?("/") && url.end_with?("/")
+          url.chop
+        else
+          url
+        end
+      rescue Addressable::URI::InvalidURIError # rubocop:disable Lint/SuppressedException; error will be reported at check time
       end
 
       private def swap_urls!

data/lib/html_proofer/cache.rb

@@ -41,7 +41,7 @@ module HTMLProofer
     end
 
     def parsed_timeframe(timeframe)
-      return nil if timeframe.nil?
+      return if timeframe.nil?
 
       time, date = timeframe.match(/(\d+)(\D)/).captures
       time = time.to_i
@@ -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)
@@ -250,7 +252,7 @@ module HTMLProofer
     SECONDS_PER_HOUR   = 3600
     SECONDS_PER_DAY    = 86400
     SECONDS_PER_WEEK   = 604800
-    SECONDS_PER_MONTH  = 2629746  # 1/12 of a gregorian year
+    SECONDS_PER_MONTH  = 2629746 # 1/12 of a gregorian year
 
     private def time_ago(measurement, unit)
       case unit
@@ -267,7 +269,8 @@ module HTMLProofer
 
     private def url_matches_type?(url, type)
       return true if type == :internal && url !~ URI_REGEXP
-      return true if type == :external && url =~ URI_REGEXP
+
+      true if type == :external && url =~ URI_REGEXP
     end
 
     private def cleaned_url(url)

data/lib/html_proofer/check/favicon.rb

@@ -17,13 +17,17 @@ 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",
+              element: @favicon,
+            )
           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",
+              element: @favicon,
+            )
           end
         else
           add_failure("no favicon provided")

data/lib/html_proofer/check/images.rb

@@ -12,31 +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})",
+            element: @img,
+          ) 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)
+            add_failure("image has no src or srcset attribute", element: @img)
           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",
+              element: @img,
+            )
           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",
+              element: @img,
+            )
           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)
+              srcset_url = HTMLProofer::Attribute::Url.new(@runner, srcset, base_url: @img.base_url, source: @img.url.source, filename: @img.url.filename, 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",
+                  element: @img,
+                )
               elsif srcset_url.remote?
                 add_to_external_urls(srcset_url.url, @img.line)
               elsif !srcset_url.exists?
-                add_failure("internal image #{srcset} does not exist", line: @img.line, content: @img.content)
+                add_failure("internal image #{srcset} does not exist", element: @img)
               end
             end
           end
@@ -44,16 +52,22 @@ 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",
+                element: @img,
+              )
             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",
+                element: @img,
+              )
             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",
+            element: @img,
+          ) if @runner.enforce_https? && @img.url.http?
         end
 
         external_urls

data/lib/html_proofer/check/links.rb

@@ -10,7 +10,7 @@ module HTMLProofer
           next if @link.ignore?
 
           if !allow_hash_href? && @link.node["href"] == "#"
-            add_failure("linking to internal hash #, which points to nowhere", line: @link.line, content: @link.content)
+            add_failure("linking to internal hash #, which points to nowhere", element: @link)
             next
           end
 
@@ -18,19 +18,21 @@ module HTMLProofer
           if blank?(@link.url.raw_attribute)
             next if allow_missing_href?
 
-            add_failure("'#{@link.node.name}' tag is missing a reference", line: @link.line, content: @link.content)
+            add_failure("'#{@link.node.name}' tag is missing a reference", element: @link)
             next
           end
 
           # is it even a valid URL?
           unless @link.url.valid?
-            add_failure("#{@link.href} is an invalid URL", line: @link.line, content: @link.content)
+            add_failure("#{@link.href} is an invalid URL", element: @link)
             next
           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",
+              element: @link,
+            )
             next
           end
 
@@ -47,7 +49,7 @@ module HTMLProofer
             next if @link.node["rel"] == "dns-prefetch"
 
             unless @link.url.path?
-              add_failure("#{@link.url.raw_attribute} is an invalid URL", line: @link.line, content: @link.content)
+              add_failure("#{@link.url.raw_attribute} is an invalid URL", element: @link)
               next
             end
 
@@ -55,8 +57,10 @@ 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",
+                element: @link,
+              )
               next
             end
 
@@ -82,23 +86,30 @@ module HTMLProofer
         when "http"
           return unless @runner.options[:enforce_https]
 
-          add_failure("#{@link.url.raw_attribute} is not an HTTPS link", line: @link.line, content: @link.content)
+          add_failure("#{@link.url.raw_attribute} is not an HTTPS link", element: @link)
         end
       end
 
       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?
-        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 no email address",
+            element: @link,
+          ) unless ignore_empty_mailto?
+        # eg., if any do not match a valid URL
+        elsif @link.url.path.split(",").any? { |email| !/#{URI::MailTo::EMAIL_REGEXP}/o.match?(email) }
+          add_failure(
+            "#{@link.url.raw_attribute} contains an invalid email address",
+            element: @link,
+          )
         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",
+          element: @link,
+        ) if @link.url.path.empty?
       end
 
       def ignore_empty_mailto?
@@ -113,13 +124,17 @@ 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}",
+            element: @link,
+          )
         elsif blank?(@link.node["integrity"])
-          add_failure("Integrity is missing in: #{@link.url.raw_attribute}", line: @link.line, content: @link.content)
+          add_failure("Integrity is missing in: #{@link.url.raw_attribute}", element: @link)
         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}",
+            element: @link,
+          )
         end
       end
 

data/lib/html_proofer/check/open_graph.rb

@@ -11,19 +11,23 @@ module HTMLProofer
 
           # does the open_graph exist?
           if missing_content?
-            add_failure("open graph has no content attribute", line: @open_graph.line, content: @open_graph.content)
+            add_failure("open graph has no content attribute", element: @open_graph)
           elsif empty_content?
-            add_failure("open graph content attribute is empty", line: @open_graph.line, content: @open_graph.content)
+            add_failure("open graph content attribute is empty", element: @open_graph)
           elsif !@open_graph.url.valid?
-            add_failure("#{@open_graph.src} is an invalid URL", line: @open_graph.line)
+            add_failure("#{@open_graph.src} is an invalid URL", element: @open_graph)
           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",
+              element: @open_graph,
+            )
           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",
+              element: @open_graph,
+            ) unless @open_graph.url.exists?
           end
         end
 

data/lib/html_proofer/check/scripts.rb

@@ -12,16 +12,20 @@ module HTMLProofer
 
           # does the script exist?
           if missing_src?
-            add_failure("script is empty and has no src attribute", line: @script.line, content: @script.content)
+            add_failure("script is empty and has no src attribute", element: @script)
           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",
+              element: @script,
+            )
           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",
+              element: @script,
+            )
           end
         end
 
@@ -34,14 +38,20 @@ 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}",
+            element: @script,
+          )
         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}",
+            element: @script,
+          )
         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}",
+            element: @script,
+          )
         end
       end
     end

data/lib/html_proofer/check.rb

@@ -24,9 +24,15 @@ module HTMLProofer
       raise NotImplementedError, "HTMLProofer::Check subclasses must implement #run"
     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)
+    def add_failure(description, element: nil, line: nil, status: nil, content: nil)
+      @failures << Failure.new(
+        @runner.current_filename,
+        short_name,
+        description,
+        line: element.nil? ? line : element.line,
+        status: status,
+        content: element.nil? ? content : element.content,
+      )
     end
 
     def short_name
@@ -39,8 +45,8 @@ module HTMLProofer
       @internal_urls[url_string] = [] if @internal_urls[url_string].nil?
 
       metadata = {
-        source: @runner.current_source,
-        filename: @runner.current_filename,
+        source: url.source,
+        filename: url.filename,
         line: line,
         base_url: base_url,
         found: false,
@@ -53,7 +59,7 @@ module HTMLProofer
 
       @external_urls[url_string] = [] if @external_urls[url_string].nil?
 
-      @external_urls[url_string] << { filename: @runner.current_filename, line: line }
+      @external_urls[url_string] << { filename: url.filename, line: line }
     end
 
     class << self

data/lib/html_proofer/configuration.rb

@@ -134,7 +134,17 @@ module HTMLProofer
 
         section(opts, "Ignore Configuration") do
           set_option(opts, "--ignore-files [FILE1,FILE2,...]") do |long_opt_symbol, list|
-            @options[long_opt_symbol] = list.nil? ? [] : list.split(",")
+            @options[long_opt_symbol] = if list.nil?
+              []
+            else
+              list.split(",").map.each do |l|
+                if l.start_with?("/") && l.end_with?("/")
+                  Regexp.new(l[1...-1])
+                else
+                  l
+                end
+              end
+            end
           end
 
           set_option(opts, "--[no-]ignore-empty-alt") do |long_opt_symbol, arg|
@@ -187,12 +197,12 @@ module HTMLProofer
             @options[long_opt_symbol] = parse_json_option("typhoeus", arg, symbolize_names: false)
           end
 
-          set_option(opts, "--hydra <CONFIG>") do |long_opt_symbol, _list|
-            @options[long_opt_symbol] = parse_json_option("hydra", arg, symbolize_names: false)
+          set_option(opts, "--hydra <CONFIG>") do |long_opt_symbol, arg|
+            @options[long_opt_symbol] = parse_json_option("hydra", arg, symbolize_names: true)
           end
 
-          set_option(opts, "--cache <CONFIG>") do |long_opt_symbol, _list|
-            @options[long_opt_symbol] = parse_json_option("cache", arg, symbolize_names: false)
+          set_option(opts, "--cache <CONFIG>") do |long_opt_symbol, arg|
+            @options[long_opt_symbol] = parse_json_option("cache", arg, symbolize_names: true)
           end
         end
 
@@ -223,8 +233,8 @@ module HTMLProofer
       arg.split(",").each_with_object({}) do |s, hsh|
         split = s.split(/(?<!\\):/, 2)
 
-        re = split[0].gsub(/\\:/, ":")
-        string = split[1].gsub(/\\:/, ":")
+        re = split[0].gsub("\\:", ":")
+        string = split[1].gsub("\\:", ":")
         hsh[Regexp.new(re)] = string
       end
     end
@@ -270,49 +280,69 @@ 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`)."],
         disable_external: ["If `true`, does not run the external link checker (default: `false`)."],
-        enforce_https: ["Fails a link if it\'s not marked as `https` (default: `true`)."],
+        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/element.rb

@@ -16,7 +16,7 @@ module HTMLProofer
       swap_attributes!
 
       @base_url = base_url
-      @url = Attribute::Url.new(runner, link_attribute, base_url: base_url)
+      @url = Attribute::Url.new(runner, link_attribute, base_url: base_url, source: @runner.current_source, filename: @runner.current_filename)
 
       @line = node.line
       @content = node.content
@@ -27,7 +27,7 @@ module HTMLProofer
     end
 
     def meta_content
-      return nil unless meta_tag?
+      return unless meta_tag?
 
       @node["content"]
     end
@@ -37,7 +37,7 @@ module HTMLProofer
     end
 
     def src
-      return nil if !img_tag? && !script_tag? && !source_tag?
+      return if !img_tag? && !script_tag? && !source_tag?
 
       @node["src"]
     end
@@ -51,7 +51,7 @@ module HTMLProofer
     end
 
     def srcset
-      return nil if !img_tag? && !source_tag?
+      return if !img_tag? && !source_tag?
 
       @node["srcset"]
     end
@@ -61,7 +61,7 @@ module HTMLProofer
     end
 
     def href
-      return nil if !a_tag? && !link_tag?
+      return if !a_tag? && !link_tag?
 
       @node["href"]
     end
@@ -82,10 +82,25 @@ module HTMLProofer
       !blank?(srcset) && srcset.split(",").size > 1
     end
 
+    # From https://github.com/sindresorhus/srcset/blob/f7c48acd7facf18e94dec47e6b96e84e0f0e69dc/index.js#LL1-L16C71
+    # This regex represents a loose rule of an “image candidate string”; see https://html.spec.whatwg.org/multipage/images.html#srcset-attribute
+    # An “image candidate string” roughly consists of the following:
+    # 1. Zero or more whitespace characters.
+    # 2. A non-empty URL that does not start or end with `,`.
+    # 3. Zero or more whitespace characters.
+    # 4. An optional “descriptor” that starts with a whitespace character.
+    # 5. Zero or more whitespace characters.
+    # 6. Each image candidate string is separated by a `,`.
+    # We intentionally implement a loose rule here so that we can perform more aggressive error handling and reporting in the below code.
+
+    IMAGE_CANDIDATE_REGEX = /\s*([^,]\S*[^,](?:\s+[^,]+)?)\s*(?:,|$)/
+
     def srcsets
-      return nil if blank?(srcset)
+      return if blank?(srcset)
 
-      srcset.split(",").map(&:strip)
+      srcset.split(IMAGE_CANDIDATE_REGEX).select.with_index do |_part, idx|
+        idx.odd?
+      end.map(&:strip)
     end
 
     def multiple_sizes?
@@ -97,7 +112,7 @@ module HTMLProofer
     end
 
     def srcsets_wo_sizes
-      return nil if blank?(srcsets)
+      return if blank?(srcsets)
 
       srcsets.map do |srcset|
         srcset.split(" ").first
@@ -118,7 +133,7 @@ module HTMLProofer
 
       attrs = @runner.options[:swap_attributes][@node.name]
 
-      return true unless blank?(attrs)
+      true unless blank?(attrs)
     end
 
     private def swap_attributes!

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
@@ -39,8 +41,8 @@ module HTMLProofer
         :red
       end
 
-      if (STDOUT_LEVELS.include?(level) && $stdout.isatty) || \
-          (STDERR_LEVELS.include?(level) && $stderr.isatty)
+      if STDOUT_LEVELS.include?(level) ||
+          STDERR_LEVELS.include?(level)
         Rainbow(message).send(color)
       else
         message

data/lib/html_proofer/reporter.rb

@@ -11,8 +11,8 @@ module HTMLProofer
     end
 
     def failures=(failures)
-      @failures = failures.group_by(&:check_name) \
-        .transform_values { |issues| issues.sort_by { |issue| [issue.path, issue.line] } } \
+      @failures = failures.group_by(&:check_name)
+        .transform_values { |issues| issues.sort_by { |issue| [issue.path, issue.line] } }
         .sort
     end
 

data/lib/html_proofer/runner.rb

@@ -6,8 +6,8 @@ module HTMLProofer
   class Runner
     include HTMLProofer::Utils
 
-    attr_reader :options, :cache, :logger, :internal_urls, :external_urls, :checked_paths, :current_check
-    attr_accessor :current_filename, :current_source, :reporter
+    attr_reader :options, :cache, :logger, :internal_urls, :external_urls, :resolved_paths, :current_check, :current_filename, :current_source
+    attr_accessor :reporter
 
     URL_TYPES = [:external, :internal].freeze
 
@@ -26,7 +26,7 @@ module HTMLProofer
 
       @before_request = []
 
-      @checked_paths = {}
+      @resolved_paths = {}
 
       @current_check = nil
       @current_source = nil
@@ -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")
@@ -166,8 +168,14 @@ module HTMLProofer
 
     def ignore_file?(file)
       @options[:ignore_files].each do |pattern|
-        return true if pattern.is_a?(String) && pattern == file
-        return true if pattern.is_a?(Regexp) && pattern =~ file
+        if pattern.is_a?(String) && pattern == file
+          @logger.log(:debug, "Ignoring #{file} because it matches #{pattern}")
+          return true
+        end
+        next unless pattern.is_a?(Regexp) && pattern.match(file)
+
+        @logger.log(:debug, "Ignoring #{file} because it matches regexp #{pattern}")
+        return true
       end
 
       false
@@ -228,7 +236,7 @@ module HTMLProofer
     end
 
     private def load_cache(type)
-      ivar = instance_variable_get("@#{type}_urls")
+      ivar = instance_variable_get(:"@#{type}_urls")
 
       existing_urls_count = @cache.size(type)
       cache_text = pluralize(existing_urls_count, "#{type} link", "#{type} links")
@@ -241,7 +249,7 @@ module HTMLProofer
 
     private def format_checks_list(checks)
       checks.map do |check|
-        check.sub(/HTMLProofer::Check::/, "")
+        check.sub("HTMLProofer::Check::", "")
       end.sort.join(", ")
     end
   end

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

@@ -29,15 +29,17 @@ module HTMLProofer
           matched_count_to_log = pluralize(matched_files.count, "reference", "references")
           @logger.log(:debug, "(#{i + 1} / #{links.count}) Internal link #{link}: Checking #{matched_count_to_log}")
           matched_files.each do |metadata|
-            url = HTMLProofer::Attribute::Url.new(@runner, link, base_url: metadata[:base_url])
-
-            @runner.current_source = metadata[:source]
-            @runner.current_filename = metadata[:filename]
-
-            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)
+            url = HTMLProofer::Attribute::Url.new(@runner, link, base_url: metadata[:base_url], source: metadata[:source], filename: metadata[:filename])
+
+            unless url.exists?
+              @failed_checks << Failure.new(
+                metadata[: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
@@ -45,6 +47,7 @@ module HTMLProofer
             hash_exists = hash_exists_for_url?(url)
             if hash_exists.nil?
               # the hash needs to be checked in the target file, we collect the url and metadata
+              target_file_path = url.resolved_path
               unless file_paths_hashes_to_check.key?(target_file_path)
                 file_paths_hashes_to_check[target_file_path] = {}
               end
@@ -55,8 +58,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(
+                metadata[: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 +84,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
@@ -91,12 +106,6 @@ module HTMLProofer
         @failed_checks
       end
 
-      private def file_exists?(absolute_path)
-        return @runner.checked_paths[absolute_path] if @runner.checked_paths.key?(absolute_path)
-
-        @runner.checked_paths[absolute_path] = File.exist?(absolute_path)
-      end
-
       # verify the hash w/o just based on the URL, w/o looking at the target file
       # => returns nil if the has could not be verified
       private def hash_exists_for_url?(url)
@@ -111,9 +120,7 @@ module HTMLProofer
         decoded_href_hash = Addressable::URI.unescape(href_hash)
         fragment_ids = [href_hash, decoded_href_hash]
         # https://www.w3.org/TR/html5/single-page.html#scroll-to-fragid
-        return true if fragment_ids.include?("top")
-
-        nil
+        true if fragment_ids.include?("top")
       end
 
       private def hash_exists_in_html?(href_hash, html)

data/lib/html_proofer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTMLProofer
-  VERSION = "5.0.0"
+  VERSION = "5.0.9"
 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

data/lib/html_proofer.rb

@@ -15,8 +15,11 @@ require "html_proofer/version"
 require "fileutils"
 
 if ENV.fetch("DEBUG", false)
-  require "awesome_print"
   require "debug"
+  begin
+    require "amazing_print"
+  rescue LoadError # rubocop:disable Lint/SuppressedException
+  end
 end
 
 module HTMLProofer

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 5.0.9
 platform: ruby
 authors:
 - Garen Torikian
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-10-08 00:00:00.000000000 Z
+date: 2024-05-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -24,6 +24,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.3'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement
@@ -38,6 +39,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.1'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
@@ -52,6 +54,22 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.13'
+  force_ruby_platform: false
+- !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'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: rainbow
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +84,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: typhoeus
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +99,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: yell
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +114,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: zeitwerk
   requirement: !ruby/object:Gem::Requirement
@@ -108,62 +129,7 @@ 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'
+  force_ruby_platform: false
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -178,48 +144,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
@@ -288,7 +212,7 @@ licenses:
 metadata:
   funding_uri: https://github.com/sponsors/gjtorikian/
   rubygems_mfa_required: 'true'
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -306,8 +230,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
-signing_key:
+rubygems_version: 3.5.3
+signing_key: 
 specification_version: 4
 summary: A set of tests to validate your HTML output. These tests check if your image
   references are legitimate, if they have alt tags, if your internal links are working,