nanoc-checking 1.0.1 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ed6936aa477c05d5d8cf11fc5958af89981b4bcd2d1b75d18a3bfea57ad32ba
-  data.tar.gz: 02feeabc06a6ef2959a070cfea91fa936734e4da9ac5e5327d0d92a8fc3ed829
+  metadata.gz: 89d7860391ed030af4773bd204ce6a94b7c76b966c42b2384efb53619f5b33f4
+  data.tar.gz: c60649c0f1f12ca8475f4c06e47348fa39dc599dcfc30c21f06e801f15f942df
 SHA512:
-  metadata.gz: 7c96031cf8552bd11e4f35113f4017481c6eb23afe95f9b5ad0446ece3c766e7aecf1600c10966826b67d18f25a2faea80ee70deb099f3bff7ed79cd0880d910
-  data.tar.gz: 1311b947dc8a91e4a42fe20470b80b2c993e8d8056c2ef2e198c7b903c9cd41d7d275aa87fda97ac05c9154aece6070e588179ee20f5d7872db8ed6927104969
+  metadata.gz: 3db4309a8fbd02942f9d8cd3db83701779ae44fd0f0e32cb92cdfc4569a3c05572ad52e309317ce911e7fd1775afe255b1c1fba4e2ff90d150716ab7f48e6d3e
+  data.tar.gz: 2e5839f8e46c65adf9721ab8eb29a09ea8734204eaa83475fb83ec66c87d0dcb440d5dc4fe06a3f476e7336f78d0af39d095d11657712f32c6473709c77e3e33

data/NEWS.md

@@ -1,5 +1,17 @@
 # nanoc-checking news
 
+## 1.0.3 (2024-03-15)
+
+Fixes:
+
+- Restore compatibility with Nanoc 4.12.20
+
+## 1.0.2 (2022-01-15)
+
+Fixes:
+
+- Ensure compatibility with latest version of Nanoc
+
 ## 1.0.1 (2021-01-01)
 
 Enhancements:

data/README.md

@@ -1,5 +1,5 @@
 # nanoc-checking
 
-This provides the `check` command and associated functionality for [Nanoc](https://nanoc.ws).
+This provides the `check` command and associated functionality for [Nanoc](https://nanoc.app).
 
-For details, see the [Checking correctness of Nanoc sites](https://nanoc.ws/doc/testing/) chapter of the Nanoc documentation.
+For details, see the [Checking correctness of Nanoc sites](https://nanoc.app/doc/testing/) chapter of the Nanoc documentation.

data/lib/nanoc/checking/check.rb

@@ -13,7 +13,7 @@ module Nanoc
     class Check < Nanoc::Core::Context
       extend DDPlugin::Plugin
 
-      DDMemoize.activate(self)
+      prepend MemoWise
 
       attr_reader :issues
 
@@ -78,13 +78,14 @@ module Nanoc
       end
 
       # @private
-      memoized def excluded_patterns
+      def excluded_patterns
         @config
           .fetch(:checks, {})
           .fetch(:all, {})
           .fetch(:exclude_files, [])
           .map { |pattern| Regexp.new(pattern) }
       end
+      memo_wise :excluded_patterns
 
       # @private
       def output_html_filenames

data/lib/nanoc/checking/checks/external_links.rb

@@ -13,7 +13,7 @@ module Nanoc
           # Find all broken external hrefs
           # TODO: de-duplicate this (duplicated in internal links check)
           filenames = output_html_filenames.reject { |f| excluded_file?(f) }
-          hrefs_with_filenames = ::Nanoc::Extra::LinkCollector.new(filenames, :external).filenames_per_href
+          hrefs_with_filenames = ::Nanoc::Checking::LinkCollector.new(filenames, :external).filenames_per_href
           results = select_invalid(hrefs_with_filenames.keys.shuffle)
 
           # Report them
@@ -21,7 +21,7 @@ module Nanoc
             filenames = hrefs_with_filenames[res.href]
             filenames.each do |filename|
               add_issue(
-                "broken reference to #{res.href}: #{res.explanation}",
+                "broken reference to <#{res.href}>: #{res.explanation}",
                 subject: filename,
               )
             end
@@ -71,7 +71,8 @@ module Nanoc
               next
             end
 
-            if /^3..$/.match?(res.code)
+            case res.code
+            when /^3..$/
               if i == 4
                 return Result.new(href, 'too many redirects')
               end
@@ -87,7 +88,7 @@ module Nanoc
               end
 
               url = URI.parse(location)
-            elsif res.code == '200'
+            when '200'
               return nil
             else
               return Result.new(href, res.code)

data/lib/nanoc/checking/checks/internal_links.rb

@@ -18,14 +18,14 @@ module Nanoc
         def run
           # TODO: de-duplicate this (duplicated in external links check)
           filenames = output_html_filenames
-          uris = ::Nanoc::Extra::LinkCollector.new(filenames, :internal).filenames_per_href
+          uris = ::Nanoc::Checking::LinkCollector.new(filenames, :internal).filenames_per_href
 
           uris.each_pair do |href, fns|
             fns.each do |filename|
               next if valid?(href, filename)
 
               add_issue(
-                "broken reference to #{href}",
+                "broken reference to <#{href}>",
                 subject: filename,
               )
             end
@@ -89,7 +89,7 @@ module Nanoc
           # FIXME: do not depend on current working directory
           origin = File.absolute_path(origin)
 
-          relative_origin = origin[@config.output_dir.size..-1]
+          relative_origin = origin[@config.output_dir.size..]
           excludes = config.fetch(:exclude_origins, [])
           excludes.any? { |pattern| Regexp.new(pattern).match(relative_origin) }
         end

data/lib/nanoc/checking/checks/mixed_content.rb

@@ -14,7 +14,7 @@ module Nanoc
 
         def run
           filenames = output_html_filenames
-          resource_uris_with_filenames = ::Nanoc::Extra::LinkCollector.new(filenames).filenames_per_resource_uri
+          resource_uris_with_filenames = ::Nanoc::Checking::LinkCollector.new(filenames).filenames_per_resource_uri
 
           resource_uris_with_filenames.each_pair do |uri, fns|
             next unless guaranteed_insecure?(uri)

data/lib/nanoc/checking/link_collector.rb

@@ -0,0 +1,132 @@
+# frozen_string_literal: true
+
+module ::Nanoc
+  module Checking
+    class LinkCollector
+      # HTML5 element attributes
+      URI_ATTRS = {
+        'a' => %i[href ping],
+        'area' => %i[href ping],
+        'audio' => %i[src],
+        'base' => %i[href],
+        'blockquote' => %i[cite],
+        'form' => %i[action],
+        'iframe' => %i[src],
+        'img' => %i[src srcset],
+        'link' => %i[href],
+        'object' => %i[data],
+        'script' => %i[src],
+        'source' => %i[src srcset],
+        'video' => %i[poster src],
+      }.freeze
+      # HTML+RDFa global URI attributes
+      GLOBAL_ATTRS = %i[about resource].freeze
+
+      def initialize(filenames, mode = nil)
+        Nanoc::Extra::JRubyNokogiriWarner.check_and_warn
+
+        @filenames = filenames
+        @filter =
+          case mode
+          when nil
+            ->(_h) { true }
+          when :external
+            ->(h) { external_href?(h) }
+          when :internal
+            ->(h) { internal_href?(h) }
+          else
+            raise ArgumentError, 'Expected mode argument to be :internal, :external or nil'
+          end
+      end
+
+      def filenames_per_href
+        grouped_filenames { |filename| hrefs_in_file(filename) }
+      end
+
+      def filenames_per_resource_uri
+        grouped_filenames { |filename| resource_uris_in_file(filename) }
+      end
+
+      def external_href?(href)
+        return false if internal_href?(href)
+
+        href =~ %r{^(//|[a-z-]+:)}
+      end
+
+      def internal_href?(href)
+        return false if href.nil?
+
+        href.start_with?('file:/')
+      end
+
+      # all links
+      def hrefs_in_file(filename)
+        uris_in_file filename, nil
+      end
+
+      # embedded resources, used by the mixed-content checker
+      def resource_uris_in_file(filename)
+        uris_in_file filename, %w[audio base form iframe img link object script source video]
+      end
+
+      private
+
+      def grouped_filenames
+        require 'nokogiri'
+        grouped_filenames = {}
+        @filenames.each do |filename|
+          yield(filename).each do |resouce_uri|
+            grouped_filenames[resouce_uri] ||= Set.new
+            grouped_filenames[resouce_uri] << filename
+          end
+        end
+        grouped_filenames
+      end
+
+      def uris_in_file(filename, tag_names)
+        uris = Set.new
+        base_uri = URI("file://#{filename}")
+        doc = Nokogiri::HTML(::File.read(filename))
+        doc.traverse do |tag|
+          next unless tag_names.nil? || tag_names.include?(tag.name)
+
+          attrs = []
+          attrs += URI_ATTRS[tag.name] unless URI_ATTRS[tag.name].nil?
+          attrs += GLOBAL_ATTRS if tag_names.nil?
+          next if attrs.nil?
+
+          attrs.each do |attr_name|
+            next if tag[attr_name].nil?
+
+            if attr_name == :srcset
+              uris = uris.merge(tag[attr_name].split(',').map { |v| v.strip.split[0].strip }.compact)
+            elsif %i[about ping resource].include?(attr_name)
+              uris = uris.merge(tag[attr_name].split.map(&:strip).compact)
+            else
+              uris << tag[attr_name.to_s]
+            end
+          end
+        end
+
+        # Strip fragment
+        uris.map! { |uri| uri.gsub(/#.*$/, '') }
+
+        # Resolve paths relative to the filename, return invalid URIs as-is
+        uris.map! do |uri|
+          if uri.start_with?('//')
+            # Don’t modify protocol-relative URLs. They’re absolute!
+            uri
+          else
+            begin
+              URI.join(base_uri, uri).to_s
+            rescue
+              uri
+            end
+          end
+        end
+
+        uris.select(&@filter)
+      end
+    end
+  end
+end

data/lib/nanoc/checking/runner.rb

@@ -70,7 +70,7 @@ module Nanoc
       def run_check_classes(classes)
         issues = run_checks(classes)
         print_issues(issues)
-        issues.empty? ? true : false
+        issues.empty?
       end
 
       def all_check_classes

data/lib/nanoc/checking/version.rb

@@ -2,6 +2,6 @@
 
 module Nanoc
   module Checking
-    VERSION = '1.0.1'
+    VERSION = '1.0.3'
   end
 end

data/lib/nanoc/checking.rb

@@ -13,6 +13,7 @@ require_relative 'checking/check'
 require_relative 'checking/checks'
 require_relative 'checking/command_runners'
 require_relative 'checking/dsl'
+require_relative 'checking/link_collector'
 require_relative 'checking/runner'
 require_relative 'checking/loader'
 require_relative 'checking/issue'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nanoc-checking
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.3
 platform: ruby
 authors:
 - Denis Defreyne
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-01 00:00:00.000000000 Z
+date: 2024-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nanoc-cli
@@ -16,40 +16,40 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.11'
+        version: '4.12'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.11.15
+        version: 4.12.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.11'
+        version: '4.12'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.11.15
+        version: 4.12.5
 - !ruby/object:Gem::Dependency
   name: nanoc-core
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.11'
+        version: '4.12'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.11.15
+        version: 4.12.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.11'
+        version: '4.12'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.11.15
+        version: 4.12.5
 description: Provides checking functionality for Nanoc
 email: denis+rubygems@denis.ws
 executables: []
@@ -74,13 +74,16 @@ files:
 - lib/nanoc/checking/commands/check.rb
 - lib/nanoc/checking/dsl.rb
 - lib/nanoc/checking/issue.rb
+- lib/nanoc/checking/link_collector.rb
 - lib/nanoc/checking/loader.rb
 - lib/nanoc/checking/runner.rb
 - lib/nanoc/checking/version.rb
-homepage: https://nanoc.ws/
+homepage: https://nanoc.app/
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
+  source_code_uri: https://github.com/nanoc/nanoc/tree/nanoc-checking-v1.0.3/nanoc-checking
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -89,14 +92,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.4
+rubygems_version: 3.5.3
 signing_key:
 specification_version: 4
 summary: Checking support for Nanoc