jekyll-url-metadata 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7cc9f88ea850dad060ca2bc836bdd131eef66684aa373048cdabd8246c16953d
-  data.tar.gz: 8b93797dc82baa3a114ea8243dde0bc9489cb085d8dc62510a2ffd583433a019
+  metadata.gz: 07f8f6a1a9019b147240b9c24ccdf325351a438430401937ee09586786318688
+  data.tar.gz: 8629259c9beed38570135fa3eebbadcd5b708e76bb33154fbae89fc54f95ddef
 SHA512:
-  metadata.gz: 15857ed5557ecb5fc3026561be968f64d98f4b39b55a9b3234b2c956b58b3434c570da016de87756911ce3d2d8c1ead5a525aacceff4c5810d60afaf9819d5ed
-  data.tar.gz: 7213d51daac4c4037a279c07f818cafdfdfa60a31fac4c324e91ed643905cae1b86202bc29cce86dc355404e4b16ab8904e7d3e35bfb484b014d77f3284ab8d8
+  metadata.gz: 7b335e1dadb10c5cc7bb228e692b6f3aba28005b720e47cbf5c34033d5713194b8e8c603c607059285bae213bfc70d8736197f8dca2f30fdc5480347858d01cb
+  data.tar.gz: 4dc6c36426d180d2e83ed8eefe75bfca97687c342651bdc2a09235a33a8c5d986b768d82a2be0d7d88c67db16bf544f63cee0074744fa8648a9f56864fcf6f0d

data/jekyll-url-metadata.gemspec

@@ -24,5 +24,6 @@ Gem::Specification.new do |spec|
   # Dependencies
   spec.add_runtime_dependency "jekyll", ">= 3.0.0"
   spec.add_runtime_dependency "nokogiri", ">= 1.10.0"
-
+  spec.add_development_dependency "rake", "~> 13.0"
+  spec.add_development_dependency "minitest", "~> 5.14"
 end

data/lib/jekyll-url-metadata/main.rb

@@ -5,8 +5,12 @@ module Jekyll
   module URLMetadata
 
     def url_config
-      @@url_config ||= @context.registers[:site].config['url_metadata'].nil? ? {} : 
-        @context.registers[:site].config['url_metadata']
+      begin
+        @@url_config ||= @context.registers[:site].config['url_metadata'].nil? ? {} : 
+          @context.registers[:site].config['url_metadata']
+      rescue
+        {}
+      end
     end
 
     def cache
@@ -14,27 +18,7 @@ module Jekyll
     end
 
     def metadata(input)
-      if !input.is_a?(String)
-        log("Expected input type \"String\". Got \"#{input.class}\".")
-        return
-      end
-
-      if input.nil? || input == ""
-        log("Empty input string.")
-        return
-      end
-
-      if !url_config["open_timeout"].nil? && !url_config["open_timeout"].is_a?(Integer)
-        log("Expected an \"Integer\" value for config \"open_timeout\". Got #{url_config["open_timeout"]}.")
-
-        return
-      end
-
-      if !url_config["read_timeout"].nil? && !url_config["read_timeout"].is_a?(Integer)
-        log("Expected an \"Integer\" value for config \"read_timeout\". Got #{url_config["read_timeout"]}.")
-
-        return
-      end
+      return if !is_input_valid(input) || !is_config_valid()
 
       # The getset() API implements a cache first strategy
       cache.getset(input) do
@@ -42,15 +26,15 @@ module Jekyll
       end
     end
 
-    def generate_hashmap(input)
+    def generate_hashmap(input, testCase = false)
       # parse HTML from URL
-      doc = Nokogiri::HTML(URI.open(input, { 
-        :open_timeout => url_config["open_timeout"].nil? ? 1 : url_config["open_timeout"],
-        :read_timeout => url_config["read_timeout"].nil? ? 1 : url_config["read_timeout"]
-      }))
-
-      if !doc
-        log("Failed to parse HTML from #{input}. Please double check for URL validity.")
+      begin
+        doc = Nokogiri::HTML(URI.open(input, { 
+          :open_timeout => url_config["open_timeout"].nil? ? 1 : url_config["open_timeout"],
+          :read_timeout => url_config["read_timeout"].nil? ? 1 : url_config["read_timeout"]
+        }))
+      rescue
+        log("Failed to parse HTML from '#{input}'. Please double check for URL validity.") if !testCase
         return
       end
 
@@ -86,8 +70,46 @@ module Jekyll
       hash
     end
 
-    def log(msg)
-      Jekyll.logger.error "URL Metadata:", msg
+    def is_input_valid(input, testCase = false)
+      if !input.is_a?(String)
+        log("Expected input type 'String'. Got '#{input.class}'.") if !testCase
+
+        return false
+      end
+
+      if input.nil? || input == ""
+        log("Empty input string.") if !testCase
+
+        return false
+      end
+
+      begin
+        if URI.parse(input).kind_of?(URI::HTTP)
+          return true
+        else
+          log("'#{input}' does not seem to be a valid URL.") if !testCase
+        end
+      rescue
+        log("'#{input}' does not seem to be a valid URL.") if !testCase
+      end
+
+      false
+    end
+
+    def is_config_valid()
+      if !url_config["open_timeout"].nil? && !url_config["open_timeout"].is_a?(Integer)
+        log("Expected an 'Integer' value for config 'open_timeout'. Got '#{url_config["open_timeout"].class}'.")
+
+        return false
+      end
+
+      if !url_config["read_timeout"].nil? && !url_config["read_timeout"].is_a?(Integer)
+        log("Expected an 'Integer' value for config 'read_timeout'. Got '#{url_config["read_timeout"].class}'.")
+
+        return false
+      end
+
+      true
     end
 
     def exists(obj)
@@ -98,6 +120,10 @@ module Jekyll
       obj.get_attribute(attr)
     end
 
+    def log(msg)
+      Jekyll.logger.error "URL Metadata:", msg
+    end
+
   end # module Jekyll
 end # module URLMetadata
 

data/lib/jekyll-url-metadata/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module URLMetadata
-    VERSION = "1.0.2"
+    VERSION = "1.0.3"
   end # module Jekyll
 end # module URLMetadata

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-url-metadata
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Gourav Khunger
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-01 00:00:00.000000000 Z
+date: 2022-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -38,6 +38,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.10.0
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.14'
 description: A plugin to expose meta data information from the head tag of a webapge
   to liquid variables just from its url string.
 email: