jekyll-remote-theme 0.3.0 → 0.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e39b39cafa24a34892141eb0d7d386988faa47a922a50195eb33458f0ec5cb18
-  data.tar.gz: 3ebfdb2d308084fabce1d92a5eba50fde22aa1be8cd742a7c44bcd9bc5f75e33
+  metadata.gz: 4c9167cea38476d7fc4ef44836e821cb07d5bcabb665e834a8df9f1cae315b70
+  data.tar.gz: 28698c0255ba7c3b9b85758e7ed52c899ffbdd49460064513d8a28a19d174b49
 SHA512:
-  metadata.gz: dfc7198186058021058cfcd85b34330fb1b8df153cc6ee2ae5ac280afac7e21a26e0c3257c8285890a676c9b14f600bdf1a8bf893997cf1370e95279353f0271
-  data.tar.gz: 93f287e3f8455eef42f66595d97e11b55aa378e2bda0e94cb644793d2ceb8429d71ab094d097d36aa036f59b4ade98c4f5af6b39cead00064939c684c003e946
+  metadata.gz: 5c9f12811ead638abbb9022f63221ef49fd6bd2f743e42d58645431aa0565946b74324131710d76ed9567adc812b316425f47e41c55ba379a4073f7812ec5b0a
+  data.tar.gz: 4ae168e2f471af7bfa0cdc39a479f3bb05fe4420d30a3036206f29b50f49f267004fa8ac4deaa4da26f887eae3f3b5e38b521cb189ed479d5262aa289d33dc11

data/lib/jekyll-remote-theme.rb

@@ -19,9 +19,9 @@ module Jekyll
     autoload :Theme,       "jekyll-remote-theme/theme"
     autoload :VERSION,     "jekyll-remote-theme/version"
 
-    CONFIG_KEY  = "remote_theme".freeze
-    LOG_KEY     = "Remote Theme:".freeze
-    TEMP_PREFIX = "jekyll-remote-theme-".freeze
+    CONFIG_KEY  = "remote_theme"
+    LOG_KEY     = "Remote Theme:"
+    TEMP_PREFIX = "jekyll-remote-theme-"
 
     def self.init(site)
       Munger.new(site).munge!

data/lib/jekyll-remote-theme/downloader.rb

@@ -3,18 +3,14 @@
 module Jekyll
   module RemoteTheme
     class Downloader
-      HOST = "https://codeload.github.com".freeze
-      PROJECT_URL = "https://github.com/benbalter/jekyll-remote-theme".freeze
-      USER_AGENT = "Jekyll Remote Theme/#{VERSION} (+#{PROJECT_URL})".freeze
+      PROJECT_URL = "https://github.com/benbalter/jekyll-remote-theme"
+      USER_AGENT = "Jekyll Remote Theme/#{VERSION} (+#{PROJECT_URL})"
       MAX_FILE_SIZE = 1 * (1024 * 1024 * 1024) # Size in bytes (1 GB)
       NET_HTTP_ERRORS = [
-        Timeout::Error, Errno::EINVAL, Errno::ECONNRESET, EOFError,
+        Timeout::Error, Errno::EINVAL, Errno::ECONNRESET, EOFError, Net::OpenTimeout,
         Net::HTTPBadResponse, Net::HTTPHeaderSyntaxError, Net::ProtocolError,
       ].freeze
 
-      attr_reader :theme
-      private :theme
-
       def initialize(theme)
         @theme = theme
       end
@@ -35,6 +31,8 @@ module Jekyll
 
       private
 
+      attr_reader :theme
+
       def zip_file
         @zip_file ||= Tempfile.new([TEMP_PREFIX, ".zip"], :binmode => true)
       end
@@ -57,6 +55,7 @@ module Jekyll
 
       def request
         return @request if defined? @request
+
         @request = Net::HTTP::Get.new zip_url.request_uri
         @request["User-Agent"] = USER_AGENT
         @request
@@ -64,11 +63,13 @@ module Jekyll
 
       def raise_unless_sucess(response)
         return if response.is_a?(Net::HTTPSuccess)
-        raise DownloadError, "#{response.code} - #{response.message}"
+
+        raise DownloadError, "#{response.code} - #{response.message} - Loading URL: #{zip_url}"
       end
 
       def enforce_max_file_size(size)
         return unless size && size > MAX_FILE_SIZE
+
         raise DownloadError, "Maximum file size of #{MAX_FILE_SIZE} bytes exceeded"
       end
 
@@ -88,8 +89,10 @@ module Jekyll
 
       # Full URL to codeload zip download endpoint for the given theme
       def zip_url
-        @zip_url ||= Addressable::URI.join(
-          HOST, "#{theme.owner}/", "#{theme.name}/", "zip/", theme.git_ref
+        @zip_url ||= Addressable::URI.new(
+          :scheme => theme.scheme,
+          :host   => "codeload.#{theme.host}",
+          :path   => [theme.owner, theme.name, "zip", theme.git_ref].join("/")
         ).normalize
       end
 

data/lib/jekyll-remote-theme/mock_gemspec.rb

@@ -9,8 +9,8 @@ module Jekyll
       extend Forwardable
       def_delegator :theme, :root, :full_gem_path
 
-      DEPENDENCY_PREFIX = %r!^\s*[a-z]+\.add_(?:runtime_)?dependency!
-      DEPENDENCY_REGEX = %r!#{DEPENDENCY_PREFIX}\(?\s*["']([a-z_-]+)["']!
+      DEPENDENCY_PREFIX = %r!^\s*[a-z]+\.add_(?:runtime_)?dependency!.freeze
+      DEPENDENCY_REGEX = %r!#{DEPENDENCY_PREFIX}\(?\s*["']([a-z_-]+)["']!.freeze
 
       def initialize(theme)
         @theme = theme

data/lib/jekyll-remote-theme/munger.rb

@@ -20,10 +20,10 @@ module Jekyll
         end
 
         Jekyll.logger.info LOG_KEY, "Using theme #{theme.name_with_owner}"
-        return theme if munged?
-
-        downloader.run
-        configure_theme
+        unless munged?
+          downloader.run
+          configure_theme
+        end
         enqueue_theme_cleanup
 
         theme
@@ -32,7 +32,7 @@ module Jekyll
       private
 
       def munged?
-        site.theme && site.theme.is_a?(Jekyll::RemoteTheme::Theme)
+        site.theme&.is_a?(Jekyll::RemoteTheme::Theme)
       end
 
       def theme
@@ -49,16 +49,16 @@ module Jekyll
 
       def configure_theme
         return unless theme
+
         site.config["theme"] = theme.name
         site.theme = theme
-        site.theme.configure_sass
+        site.theme.configure_sass if site.theme.respond_to?(:configure_sass)
         site.send(:configure_include_paths)
         site.plugin_manager.require_theme_deps
       end
 
       def enqueue_theme_cleanup
         at_exit do
-          return unless munged? && downloader.downloaded?
           Jekyll.logger.debug LOG_KEY, "Cleaning up #{theme.root}"
           FileUtils.rm_rf theme.root
         end

data/lib/jekyll-remote-theme/theme.rb

@@ -3,10 +3,10 @@
 module Jekyll
   module RemoteTheme
     class Theme < Jekyll::Theme
-      OWNER_REGEX = %r!(?<owner>[a-z0-9\-]+)!i
-      NAME_REGEX  = %r!(?<name>[a-z0-9\._\-]+)!i
-      REF_REGEX   = %r!@(?<ref>[a-z0-9\._\-]+)!i # May be a branch, tag, or commit
-      THEME_REGEX = %r!\A#{OWNER_REGEX}/#{NAME_REGEX}(?:#{REF_REGEX})?\z!i
+      OWNER_REGEX = %r!(?<owner>[a-z0-9\-]+)!i.freeze
+      NAME_REGEX  = %r!(?<name>[a-z0-9\._\-]+)!i.freeze
+      REF_REGEX   = %r!@(?<ref>[a-z0-9\._\-]+)!i.freeze # May be a branch, tag, or commit
+      THEME_REGEX = %r!\A#{OWNER_REGEX}/#{NAME_REGEX}(?:#{REF_REGEX})?\z!i.freeze
 
       # Initializes a new Jekyll::RemoteTheme::Theme
       #
@@ -14,6 +14,10 @@ module Jekyll
       #
       # 1. owner/theme-name - a GitHub owner + theme-name string
       # 2. owner/theme-name@git_ref - a GitHub owner + theme-name + Git ref string
+      # 3. http[s]://github.<yourEnterprise>.com/owner/theme-name
+      # - An enterprise GitHub instance + a GitHub owner + a theme-name string
+      # 4. http[s]://github.<yourEnterprise>.com/owner/theme-name@git_ref
+      # - An enterprise GitHub instance + a GitHub owner + a theme-name + Git ref string
       def initialize(raw_theme)
         @raw_theme = raw_theme.to_s.downcase.strip
         super(@raw_theme)
@@ -27,17 +31,27 @@ module Jekyll
         theme_parts[:owner]
       end
 
+      def host
+        uri&.host
+      end
+
+      def scheme
+        uri&.scheme
+      end
+
       def name_with_owner
         [owner, name].join("/")
       end
       alias_method :nwo, :name_with_owner
 
       def valid?
-        theme_parts && name && owner
+        return false unless uri && theme_parts && name && owner
+
+        host && valid_hosts.include?(host)
       end
 
       def git_ref
-        theme_parts[:ref] || "master"
+        theme_parts[:ref] || "HEAD"
       end
 
       def root
@@ -45,19 +59,43 @@ module Jekyll
       end
 
       def inspect
-        "#<Jekyll::RemoteTheme::Theme owner=\"#{owner}\" name=\"#{name}\"" \
+        "#<Jekyll::RemoteTheme::Theme host=\"#{host}\" owner=\"#{owner}\" name=\"#{name}\"" \
         " ref=\"#{git_ref}\" root=\"#{root}\">"
       end
 
       private
 
+      def uri
+        return @uri if defined? @uri
+
+        @uri = if THEME_REGEX.match?(@raw_theme)
+                 Addressable::URI.new(
+                   :scheme => "https",
+                   :host   => "github.com",
+                   :path   => @raw_theme
+                 )
+               else
+                 Addressable::URI.parse @raw_theme
+               end
+      rescue Addressable::URI::InvalidURIError
+        @uri = nil
+      end
+
       def theme_parts
-        @theme_parts ||= @raw_theme.match(THEME_REGEX)
+        @theme_parts ||= uri.path[1..-1].match(THEME_REGEX) if uri
       end
 
       def gemspec
         @gemspec ||= MockGemspec.new(self)
       end
+
+      def valid_hosts
+        @valid_hosts ||= [
+          "github.com",
+          ENV["PAGES_GITHUB_HOSTNAME"],
+          ENV["GITHUB_HOSTNAME"],
+        ].compact.to_set
+      end
     end
   end
 end

data/lib/jekyll-remote-theme/version.rb

@@ -2,6 +2,6 @@
 
 module Jekyll
   module RemoteTheme
-    VERSION = "0.3.0".freeze
+    VERSION = "0.4.3"
   end
 end

metadata

@@ -1,36 +1,82 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-remote-theme
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.3
 platform: ruby
 authors:
 - Ben Balter
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-05-14 00:00:00.000000000 Z
+date: 2021-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: jekyll
+  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.5'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: jekyll
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: jekyll-sass-converter
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: rubyzip
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.3.0
     - - "<"
       - !ruby/object:Gem::Version
         version: '3.0'
@@ -40,7 +86,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.3.0
     - - "<"
       - !ruby/object:Gem::Version
         version: '3.0'
@@ -72,6 +118,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: kramdown-parser-gfm
+  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: pry
   requirement: !ruby/object:Gem::Requirement
@@ -106,20 +166,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.49.0
+        version: '0.71'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
-    - - ">="
+        version: '0.71'
+- !ruby/object:Gem::Dependency
+  name: rubocop-jekyll
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.49.0
+        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: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -134,7 +202,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
-description: 
+description:
 email:
 - ben.balter@github.com
 executables: []
@@ -151,7 +219,7 @@ homepage: https://github.com/benbalter/jekyll-remote-theme
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -159,16 +227,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubygems_version: 3.2.14
+signing_key:
 specification_version: 4
 summary: Jekyll plugin for building Jekyll sites with any GitHub-hosted theme
 test_files: []