github-pages-health-check 1.14.0 → 1.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ebf0365b555a468e1785cdd41997442c825385a0a30f77474d8af65c404b6cf
-  data.tar.gz: 3b7082f91aea5dd278c5f75eb2f4d8bdf93ed64e3f40b578930739e18c6d596c
+  metadata.gz: e269d0849d9754d658dde60478ee1bb1a87913fd6e10cc79cc6cab834b899946
+  data.tar.gz: 83a05c02f53f262c02bcd7d1bf00801b7860fa878c32251fc24006f5dcf454ad
 SHA512:
-  metadata.gz: fa4bc9f48ecf20b5541a9eb075df3d119707cc8c00777ae2718fe2955a194d1ab9e50027a3c48045a7de001171195f75068435b8709b13ff0a7560d098a7ef7c
-  data.tar.gz: d918ba62878f48d6539d1feead74b7da1bfc74d28f95263140eef339e9067980c22aaead6a894880470761686c95679f399d867ce0ec8afa429b3ed5b5fe2d79
+  metadata.gz: 765bb285569e16d11123448b7a2bfcc31c6317293dab5b5a64c6f8af2c9925f5e000b9885273fb1b8c1b42b6db94340dde7292f7227c46b528e56ae0631916ab
+  data.tar.gz: ff8e0af41eb9e43bbe126a969a66fc6ebe704ed196f6a8d2230572ddb52ccfd089af823ddaf016600d117d343ce589f4e492c2b5148926d1c1eae83444900163

data/.dockerignore

@@ -0,0 +1 @@
+Gemfile.lock

data/.github/dependabot.yml

@@ -0,0 +1,15 @@
+version: 2
+updates:
+- package-ecosystem: bundler
+  directory: "/"
+  schedule:
+    interval: daily
+    time: "10:00"
+    timezone: Europe/Vienna
+  pull-request-branch-name:
+    separator: "-"
+  open-pull-requests-limit: 99
+  allow:
+  - dependency-type: direct
+  - dependency-type: indirect
+  rebase-strategy: disabled

data/.github/workflows/push-cibuild.yml

@@ -0,0 +1,18 @@
+on: push
+name: cibuild on push
+jobs:
+  build:
+    name: "GitHub Pages Health Check Tests"
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby:
+          - 2.5
+          - 2.6
+          - 2.7
+    steps:
+    - uses: actions/checkout@master
+    - name: script/cibuild-docker
+      run: script/cibuild-docker
+      env:
+        RUBY_VERSION: ${{ matrix.ruby }}

data/.rubocop.yml

@@ -17,7 +17,7 @@
 #
 
 AllCops:
-  TargetRubyVersion: 2.2
+  TargetRubyVersion: 2.6
   Exclude:
     - 'bin/**/*'
     - 'script/**/*'
@@ -30,10 +30,6 @@ Layout/EndAlignment:
 Lint/UnreachableCode:
   Severity: error
 
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
-  Severity: error
-
 Style/StringLiteralsInInterpolation:
   EnforcedStyle: double_quotes
 
@@ -41,10 +37,10 @@ Style/HashSyntax:
   EnforcedStyle: hash_rockets
   Severity: error
 
-Layout/AlignHash:
+Layout/HashAlignment:
   SupportedLastArgumentHashStyles: always_ignore
 
-Layout/AlignParameters:
+Layout/ParameterAlignment:
   Enabled: false # This is usually true, but we often want to roll back to
                  # the start of a line.
 
@@ -55,12 +51,7 @@ Style/Attr:
 Style/ClassAndModuleChildren:
   Enabled: false # module X<\n>module Y is just as good as module X::Y.
 
-Style/PercentLiteralDelimiters:
-  PreferredDelimiters:
-    '%w': '{}'
-    '%r': '{}'
-
-Metrics/LineLength:
+Layout/LineLength:
   Max: 90
   Severity: warning
   Exclude:
@@ -99,13 +90,12 @@ Naming/FileName: #Rubocop doesn't like the Git*H*ub namespace
   Enabled: false
 
 Metrics/ParameterLists: { Max: 4 }
-Metrics/AbcSize: { Max: 20 }
 
-Layout/IndentHash: { EnforcedStyle: consistent }
+Layout/FirstHashElementIndentation: { EnforcedStyle: consistent }
 Layout/MultilineMethodCallIndentation: { EnforcedStyle: indented }
 Layout/MultilineOperationIndentation: { EnforcedStyle: indented }
 Layout/FirstParameterIndentation: { EnforcedStyle: consistent }
-Layout/IndentArray: { EnforcedStyle: consistent }
+Layout/FirstArrayElementIndentation: { EnforcedStyle: consistent }
 Layout/ExtraSpacing: { AllowForAlignment: true }
 Style/SignalException: { EnforcedStyle: only_raise }
 Style/StringLiterals: { EnforcedStyle: double_quotes }
@@ -155,3 +145,10 @@ Style/FrozenStringLiteralComment:
 
 Gemspec/RequiredRubyVersion:
   Enabled: false
+
+Style/HashEachMethods:
+  Enabled: false
+Style/HashTransformKeys:
+  Enabled: false
+Style/HashTransformValues:
+  Enabled: false

data/.ruby-version

@@ -0,0 +1 @@
+2.7.1

data/.travis.yml

@@ -1,12 +1,11 @@
 language: ruby
 rvm:
-  - 2.2
-  - 2.3
-  - 2.4
   - 2.5
+  - 2.6
+  - 2.7
 
 before_install:
-  - gem install bundler
+  - gem update --system
 
 script: "script/cibuild"
 

data/Dockerfile

@@ -0,0 +1,18 @@
+ARG RUBY_VERSION
+FROM ruby:$RUBY_VERSION-slim
+RUN set -ex \
+  && gem update --system --silent --quiet \
+  && apt-get update -y \
+  && apt-get upgrade -y \
+  && apt-get install -y \
+    build-essential \
+    git \
+    libcurl4-openssl-dev \
+  && apt-get clean
+WORKDIR /app/github-pages-health-check
+COPY Gemfile .
+COPY github-pages-health-check.gemspec .
+COPY lib/github-pages-health-check/version.rb lib/github-pages-health-check/version.rb
+RUN bundle install
+COPY . .
+ENTRYPOINT [ "/bin/bash" ]

data/Gemfile

@@ -2,4 +2,13 @@
 
 source "https://rubygems.org"
 
+group :development do
+  gem "dotenv", "~> 2.7"
+  gem "gem-release", "~> 2.1"
+  gem "pry", "~> 0.10"
+  gem "rspec", "~> 3.0"
+  gem "rubocop", "~> 0.52"
+  gem "webmock", "~> 3.8"
+end
+
 gemspec

data/config/cloudflare-ips.txt

@@ -1,14 +1,15 @@
+173.245.48.0/20
 103.21.244.0/22
 103.22.200.0/22
 103.31.4.0/22
-104.16.0.0/12
-108.162.192.0/18
-131.0.72.0/22
 141.101.64.0/18
-162.158.0.0/15
-172.64.0.0/13
-173.245.48.0/20
-188.114.96.0/20
+108.162.192.0/18
 190.93.240.0/20
+188.114.96.0/20
 197.234.240.0/22
 198.41.128.0/17
+162.158.0.0/15
+172.64.0.0/13
+131.0.72.0/22
+104.16.0.0/13
+104.24.0.0/14

data/config/fastly-ips.txt

@@ -4,13 +4,14 @@
 103.245.222.0/23
 103.245.224.0/24
 104.156.80.0/20
+146.75.0.0/16
 151.101.0.0/16
 157.52.64.0/18
+167.82.0.0/17
+167.82.128.0/20
+167.82.160.0/20
+167.82.224.0/20
 172.111.64.0/18
 185.31.16.0/22
 199.27.72.0/21
-199.232.0.0/16
-202.21.128.11/32
-202.21.128.12/32
-203.57.145.11/32
-203.57.145.12/32
+199.232.0.0/16

data/github-pages-health-check.gemspec

@@ -19,13 +19,6 @@ Gem::Specification.new do |s|
   s.add_dependency("addressable", "~> 2.3")
   s.add_dependency("dnsruby", "~> 1.60")
   s.add_dependency("octokit", "~> 4.0")
-  s.add_dependency("public_suffix", "~> 3.0")
+  s.add_dependency("public_suffix", ">= 2.0.2", "< 5.0")
   s.add_dependency("typhoeus", "~> 1.3")
-
-  s.add_development_dependency("dotenv", "~> 1.0")
-  s.add_development_dependency("gem-release", "~> 0.7")
-  s.add_development_dependency("pry", "~> 0.10")
-  s.add_development_dependency("rspec", "~> 3.0")
-  s.add_development_dependency("rubocop", "~> 0.52")
-  s.add_development_dependency("webmock", "~> 1.21")
 end

data/lib/github-pages-health-check.rb

@@ -37,9 +37,9 @@ module GitHubPages
     # DNS and HTTP timeout, in seconds
     TIMEOUT = 7
 
-    HUMAN_NAME = "GitHub Pages Health Check".freeze
-    URL = "https://github.com/github/pages-health-check".freeze
-    USER_AGENT = "Mozilla/5.0 (compatible; #{HUMAN_NAME}/#{VERSION}; +#{URL})".freeze
+    HUMAN_NAME = "GitHub Pages Health Check"
+    URL = "https://github.com/github/pages-health-check"
+    USER_AGENT = "Mozilla/5.0 (compatible; #{HUMAN_NAME}/#{VERSION}; +#{URL})"
 
     TYPHOEUS_OPTIONS = {
       :followlocation => true,

data/lib/github-pages-health-check/caa.rb

@@ -9,7 +9,7 @@ module GitHubPages
     class CAA
       attr_reader :host, :error, :nameservers
 
-      def initialize(host, nameservers: nil)
+      def initialize(host:, nameservers: :default)
         raise ArgumentError, "host cannot be nil" if host.nil?
 
         @host = host
@@ -24,24 +24,30 @@ module GitHubPages
       def lets_encrypt_allowed?
         return false if errored?
         return true unless records_present?
+
         records.any? { |r| r.property_value == "letsencrypt.org" }
       end
 
       def records_present?
         return false if errored?
+
         records && !records.empty?
       end
 
       def records
-        @records ||= begin
-          get_caa_records(host) | get_caa_records(host.split(".").drop(1).join("."))
-        end
+        return @records if defined?(@records)
+
+        @records = get_caa_records(host)
+        @records = get_caa_records(parent_host) if @records.nil? || @records.empty?
+
+        @records
       end
 
       private
 
       def get_caa_records(domain)
         return [] if domain.nil?
+
         query(domain).select { |r| issue_caa_record?(r) }
       end
 
@@ -59,6 +65,10 @@ module GitHubPages
       def resolver(domain)
         GitHubPages::HealthCheck::Resolver.new(domain, :nameservers => nameservers)
       end
+
+      def parent_host
+        host.split(".").drop(1).join(".")
+      end
     end
   end
 end

data/lib/github-pages-health-check/domain.rb

@@ -100,44 +100,52 @@ module GitHubPages
         @host = normalize_host(host)
         @nameservers = nameservers
         @resolver = GitHubPages::HealthCheck::Resolver.new(self.host,
-          :nameservers => nameservers)
+                                                           :nameservers => nameservers)
       end
 
       # Runs all checks, raises an error if invalid
+      # rubocop:disable Metrics/AbcSize
       def check!
-        raise Errors::InvalidDomainError, :domain => self unless valid_domain?
-        raise Errors::InvalidDNSError, :domain => self    unless dns_resolves?
-        raise Errors::DeprecatedIPError, :domain => self if deprecated_ip?
+        raise Errors::InvalidDomainError.new :domain => self unless valid_domain?
+        raise Errors::InvalidDNSError.new :domain => self    unless dns_resolves?
+        raise Errors::DeprecatedIPError.new :domain => self  if deprecated_ip?
         return true if proxied?
-        raise Errors::InvalidARecordError, :domain => self    if invalid_a_record?
-        raise Errors::InvalidCNAMEError, :domain => self      if invalid_cname?
-        raise Errors::InvalidAAAARecordError, :domain => self if invalid_aaaa_record?
-        raise Errors::NotServedByPagesError, :domain => self  unless served_by_pages?
+        raise Errors::InvalidARecordError.new :domain => self    if invalid_a_record?
+        raise Errors::InvalidCNAMEError.new :domain => self      if invalid_cname?
+        raise Errors::InvalidAAAARecordError.new :domain => self if invalid_aaaa_record?
+        raise Errors::NotServedByPagesError.new :domain => self  unless served_by_pages?
+
         true
       end
+      # rubocop:enable Metrics/AbcSize
 
       def deprecated_ip?
         return @deprecated_ip if defined? @deprecated_ip
+
         @deprecated_ip = (valid_domain? && a_record? && old_ip_address?)
       end
 
       def invalid_aaaa_record?
         return @invalid_aaaa_record if defined? @invalid_aaaa_record
+
         @invalid_aaaa_record = (valid_domain? && should_be_a_record? &&
                                 aaaa_record_present?)
       end
 
       def invalid_a_record?
         return @invalid_a_record if defined? @invalid_a_record
+
         @invalid_a_record = (valid_domain? && a_record? && !should_be_a_record?)
       end
 
       def invalid_cname?
         return @invalid_cname if defined? @invalid_cname
+
         @invalid_cname = begin
           return false unless valid_domain?
           return false if github_domain? || apex_domain?
           return true  if cname_to_pages_dot_github_dot_com? || cname_to_fastly?
+
           !cname_to_github_user_domain? && should_be_cname_record?
         end
       end
@@ -146,8 +154,11 @@ module GitHubPages
       # Used as an escape hatch to prevent false positives on DNS checkes
       def valid_domain?
         return @valid if defined? @valid
+
         unicode_host = Addressable::IDNA.to_unicode(host)
-        @valid = PublicSuffix.valid?(unicode_host, :default_rule => nil)
+        @valid = PublicSuffix.valid?(unicode_host,
+                                     :default_rule => nil,
+                                     :ignore_private => true)
       end
 
       # Is this domain an apex domain, meaning a CNAME would be innapropriate
@@ -161,7 +172,9 @@ module GitHubPages
         # E.g. PublicSuffix.domain("blog.digital.gov.uk") # => "digital.gov.uk"
         # For apex-level domain names, DNS providers do not support CNAME records.
         unicode_host = Addressable::IDNA.to_unicode(host)
-        PublicSuffix.domain(unicode_host) == unicode_host
+        PublicSuffix.domain(unicode_host,
+                            :default_rule => nil,
+                            :ignore_private => true) == unicode_host
       end
 
       # Should the domain use an A record?
@@ -181,6 +194,7 @@ module GitHubPages
       # Are any of the domain's A records pointing elsewhere?
       def non_github_pages_ip_present?
         return unless dns?
+
         a_records = dns.select { |answer| answer.type == Dnsruby::Types::A }
 
         a_records.any? { |answer| !github_pages_ip?(answer.address.to_s) }
@@ -256,6 +270,7 @@ module GitHubPages
         return false if cname_to_github_user_domain?
         return false if cname_to_pages_dot_github_dot_com?
         return false if cname_to_fastly? || fastly_ip?
+
         served_by_pages?
       end
 
@@ -270,9 +285,11 @@ module GitHubPages
       def dns
         return @dns if defined? @dns
         return unless valid_domain?
+
         @dns = Timeout.timeout(TIMEOUT) do
           GitHubPages::HealthCheck.without_warnings do
             next if host.nil?
+
             REQUESTED_RECORD_TYPES
               .map { |type| resolver.query(type) }
               .flatten.uniq
@@ -300,11 +317,13 @@ module GitHubPages
       # Is this domain's first response an A record?
       def a_record?
         return unless dns?
+
         dns.first.type == Dnsruby::Types::A
       end
 
       def aaaa_record_present?
         return unless dns?
+
         dns.any? { |answer| answer.type == Dnsruby::Types::AAAA }
       end
 
@@ -312,6 +331,7 @@ module GitHubPages
       def cname_record?
         return unless dns?
         return false unless cname
+
         cname.valid_domain?
       end
       alias cname? cname_record?
@@ -319,12 +339,15 @@ module GitHubPages
       # The domain to which this domain's CNAME resolves
       # Returns nil if the domain is not a CNAME
       def cname
-        return unless dns.first.type == Dnsruby::Types::CNAME
-        @cname ||= Domain.new(dns.first.cname.to_s)
+        cnames = dns.take_while { |answer| answer.type == Dnsruby::Types::CNAME }
+        return if cnames.empty?
+
+        @cname ||= Domain.new(cnames.last.cname.to_s)
       end
 
       def mx_records_present?
         return unless dns?
+
         dns.any? { |answer| answer.type == Dnsruby::Types::MX }
       end
 
@@ -337,7 +360,7 @@ module GitHubPages
           return true if response.headers["Server"] == "GitHub.com"
 
           # Typhoeus mangles the case of the header, compare insensitively
-          response.headers.any? { |k, _v| k =~ /X-GitHub-Request-Id/i }
+          response.headers.any? { |k, _v| k.downcase == "x-github-request-id" }
         end
       end
 
@@ -361,27 +384,40 @@ module GitHubPages
       # Does this domain redirect HTTP requests to HTTPS?
       def enforces_https?
         return false unless https? && http_response.headers["Location"]
+
         redirect = Addressable::URI.parse(http_response.headers["Location"])
         redirect.scheme == "https" && redirect.host == host
       end
 
       # Can an HTTPS certificate be issued for this domain?
       def https_eligible?
-        (cname_to_github_user_domain? || pointed_to_github_pages_ip?) &&
-          !aaaa_record_present? && !non_github_pages_ip_present? &&
-          caa.lets_encrypt_allowed?
+        # Can't have any IP's which aren't GitHub's present.
+        return false if non_github_pages_ip_present?
+        # Can't have any AAAA records present
+        return false if aaaa_record_present?
+        # Must be a CNAME or point to our IPs.
+
+        # Only check the one domain if a CNAME. Don't check the parent domain.
+        return true if cname_to_github_user_domain?
+
+        # Check CAA records for the full domain and its parent domain.
+        pointed_to_github_pages_ip? && caa.lets_encrypt_allowed?
       end
 
       # Any errors querying CAA records
       def caa_error
         return nil unless caa.errored?
+
         caa.error.class.name
       end
 
       private
 
       def caa
-        @caa ||= GitHubPages::HealthCheck::CAA.new(host, :nameservers => nameservers)
+        @caa ||= GitHubPages::HealthCheck::CAA.new(
+          :host => cname&.host || host,
+          :nameservers => nameservers
+        )
       end
 
       # The domain's response to HTTP(S) requests, following redirects

data/lib/github-pages-health-check/error.rb

@@ -3,8 +3,8 @@
 module GitHubPages
   module HealthCheck
     class Error < StandardError
-      DOCUMENTATION_BASE = "https://help.github.com".freeze
-      DOCUMENTATION_PATH = "/categories/github-pages-basics/".freeze
+      DOCUMENTATION_BASE = "https://help.github.com"
+      DOCUMENTATION_PATH = "/categories/github-pages-basics/"
       LOCAL_ONLY = false # Error is only used when running locally
 
       attr_reader :repository, :domain

data/lib/github-pages-health-check/errors.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-Dir[File.expand_path("errors/*_error.rb", __dir__)].each do |f|
+Dir[File.expand_path("errors/*_error.rb", __dir__)].sort.each do |f|
   require f
 end
 

data/lib/github-pages-health-check/errors/build_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class BuildError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/troubleshooting-jekyll-builds/".freeze
+        DOCUMENTATION_PATH = "/articles/troubleshooting-jekyll-builds/"
         LOCAL_ONLY = true
       end
     end

data/lib/github-pages-health-check/errors/deprecated_ip_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class DeprecatedIPError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           <<-MSG

data/lib/github-pages-health-check/errors/invalid_a_record_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class InvalidARecordError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           <<-MSG

data/lib/github-pages-health-check/errors/invalid_aaaa_record_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class InvalidAAAARecordError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           <<-MSG

data/lib/github-pages-health-check/errors/invalid_cname_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class InvalidCNAMEError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           <<-MSG

data/lib/github-pages-health-check/errors/invalid_dns_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class InvalidDNSError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           "Domain's DNS record could not be retrieved"

data/lib/github-pages-health-check/errors/invalid_domain_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class InvalidDomainError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           "Domain is not a valid domain"

data/lib/github-pages-health-check/errors/not_served_by_pages_error.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     module Errors
       class NotServedByPagesError < GitHubPages::HealthCheck::Error
-        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/".freeze
+        DOCUMENTATION_PATH = "/articles/setting-up-a-custom-domain-with-github-pages/"
 
         def message
           "Domain does not resolve to the GitHub Pages server"

data/lib/github-pages-health-check/printer.rb

@@ -4,7 +4,7 @@ module GitHubPages
   module HealthCheck
     class Printer
       PRETTY_LEFT_WIDTH = 11
-      PRETTY_JOINER = " | ".freeze
+      PRETTY_JOINER = " | "
 
       attr_reader :health_check
 

data/lib/github-pages-health-check/repository.rb

@@ -5,7 +5,7 @@ module GitHubPages
     class Repository < Checkable
       attr_reader :name, :owner
 
-      REPO_REGEX = %r{\A[a-z0-9_\-]+/[a-z0-9_\-\.]+\z}i
+      REPO_REGEX = %r{\A[a-z0-9_\-]+/[a-z0-9_\-\.]+\z}i.freeze
 
       HASH_METHODS = %i[
         name_with_owner built? last_built build_duration build_error
@@ -15,6 +15,7 @@ module GitHubPages
         unless name_with_owner.match(REPO_REGEX)
           raise Errors::InvalidRepositoryError
         end
+
         parts = name_with_owner.split("/")
         @owner = parts.first
         @name  = parts.last
@@ -28,6 +29,7 @@ module GitHubPages
 
       def check!
         raise Errors::BuildError.new(:repository => self), build_error unless built?
+
         true
       end
 
@@ -45,15 +47,16 @@ module GitHubPages
       alias reason build_error
 
       def build_duration
-        last_build && last_build.duration
+        last_build&.duration
       end
 
       def last_built
-        last_build && last_build.updated_at
+        last_build&.updated_at
       end
 
       def domain
         return if cname.nil?
+
         @domain ||= GitHubPages::HealthCheck::Domain.redundant(cname)
       end
 
@@ -61,6 +64,7 @@ module GitHubPages
 
       def client
         raise Errors::MissingAccessTokenError if @access_token.nil?
+
         @client ||= Octokit::Client.new(:access_token => @access_token)
       end
 

data/lib/github-pages-health-check/resolver.rb

@@ -4,10 +4,10 @@ module GitHubPages
   module HealthCheck
     class Resolver
       DEFAULT_RESOLVER_OPTIONS = {
-        :retry_times   => 2,
+        :retry_times => 2,
         :query_timeout => 5,
-        :dnssec        => false,
-        :do_caching    => false
+        :dnssec => false,
+        :do_caching => false
       }.freeze
       PUBLIC_NAMESERVERS = %w(
         8.8.8.8
@@ -43,16 +43,16 @@ module GitHubPages
                         self.class.default_resolver
                       when :authoritative
                         Dnsruby::Resolver.new(DEFAULT_RESOLVER_OPTIONS.merge(
-                          :nameservers => authoritative_nameservers
-                        ))
+                                                :nameservers => authoritative_nameservers
+                                              ))
                       when :public
                         Dnsruby::Resolver.new(DEFAULT_RESOLVER_OPTIONS.merge(
-                          :nameservers => PUBLIC_NAMESERVERS
-                        ))
+                                                :nameservers => PUBLIC_NAMESERVERS
+                                              ))
                       when Array
                         Dnsruby::Resolver.new(DEFAULT_RESOLVER_OPTIONS.merge(
-                          :nameservers => nameservers
-                        ))
+                                                :nameservers => nameservers
+                                              ))
                       else
                         raise "Invalid nameserver type: #{nameservers.inspect}"
                       end

data/lib/github-pages-health-check/version.rb

@@ -2,6 +2,6 @@
 
 module GitHubPages
   module HealthCheck
-    VERSION = "1.14.0".freeze
+    VERSION = "1.17.1"
   end
 end

data/script/cibuild

@@ -4,7 +4,7 @@ set -ex
 
 script/bootstrap
 
-script/test
+script/test $@
 script/check-cdn-ips
 bundle exec script/check www.parkermoore.de | grep 'valid?: true'
 bundle exec script/check ben.balter.com | grep 'valid?: true'

data/script/cibuild-docker

@@ -0,0 +1,11 @@
+#!/bin/bash
+
+: ${RUBY_VERSION:="2.7"}
+docker build -t github-pages-health-check --build-arg RUBY_VERSION=$RUBY_VERSION .
+if [ -n "$DEBUG" ]; then
+  # Run a shell.
+  docker run -it --rm -v $(pwd):/app/github-pages-health-check github-pages-health-check
+else
+  # Run CI
+  docker run --rm github-pages-health-check script/cibuild --profile --fail-fast
+fi

data/script/test

@@ -3,4 +3,4 @@
 set -e
 
 bundle exec rspec $@
-script/fmt $@
+script/fmt

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: github-pages-health-check
 version: !ruby/object:Gem::Version
-  version: 1.14.0
+  version: 1.17.1
 platform: ruby
 authors:
 - GitHub, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-17 00:00:00.000000000 Z
+date: 2021-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -56,16 +56,22 @@ dependencies:
   name: public_suffix
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.2
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.2
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: typhoeus
   requirement: !ruby/object:Gem::Requirement
@@ -80,100 +86,21 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
-- !ruby/object:Gem::Dependency
-  name: dotenv
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-- !ruby/object:Gem::Dependency
-  name: gem-release
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.7'
-- !ruby/object:Gem::Dependency
-  name: pry
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.10'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.10'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.52'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.52'
-- !ruby/object:Gem::Dependency
-  name: webmock
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.21'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.21'
 description: Checks your GitHub Pages site for commons DNS configuration issues.
 email: support@github.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".dockerignore"
+- ".github/dependabot.yml"
+- ".github/workflows/push-cibuild.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
+- ".ruby-version"
 - ".travis.yml"
+- Dockerfile
 - Gemfile
 - LICENSE.md
 - README.md
@@ -209,6 +136,7 @@ files:
 - script/check
 - script/check-cdn-ips
 - script/cibuild
+- script/cibuild-docker
 - script/console
 - script/fmt
 - script/release
@@ -233,8 +161,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Checks your GitHub Pages site for commons DNS configuration issues