jekyll-embed-urls 0.5.6 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e6e72a30b43d661e9c852d548f48ba8f600cbbde41e6873489cf9ceb3d4435c
-  data.tar.gz: dd437ba4bbfa32d8e9c26792a3547672ae4197d8b23029b538d3a1ca814e64b7
+  metadata.gz: '009ec95cb733855d621b7b063e4922c79873296fdebd99f16ad6f2856854a7af'
+  data.tar.gz: e56ca532db963a099a4c4c6950c4382ca25d83c5102703b8bf2d66fd98d8f2a9
 SHA512:
-  metadata.gz: 9c6cb5356446f930287dd1feca9ac024db143a0f3bf531dc19f94e6e7b4a2602125a2b20bafb1029da46ad15c4cdfcbb4c2e783967f6bba16e1b21310c496557
-  data.tar.gz: b98314a98b042a9c7cf1f9cfbeacce6662d11b6c5ec8d6d85e0d96ea3f2bc32d0e0bcea8cf3fd387b7872fcbeb0555d2873ff242dc84f8e503e40a2374f23140
+  metadata.gz: d60e0cfb3395976201a2b9b4174aaf1b888ef88fede4c802302002787471a0d8268c89d8d1cb447fa24bcebee7ac5f4ce83865b5770e0fe5ca38cd93ebb7e8e4
+  data.tar.gz: 4310e1aa4409b2724e8dfc9b68ecc96e4eabe83709a19cb7e5d3745a895b9e7bb32af78dece340f6c4e37807813582cdb07517253a40f739c7206db03db606f3

data/lib/jekyll/embed/cache.rb

@@ -10,7 +10,7 @@ module Jekyll
 
       def read(key)
         self[key]
-      rescue
+      rescue StandardError
         nil
       end
     end

data/lib/jekyll/embed/content.rb

@@ -5,7 +5,7 @@ require 'cgi'
 module Jekyll
   class Embed
     class Content
-      URL_RE = /(<p[^>]*>[\s\n]*(<a[^>]*>)?[\s\n]*(https?:\/\/[^<\s\n]+)[\s\n]*(<\/a>)?[\s\n]*<\/p>)/m.freeze
+      URL_RE = %r{(<p[^>]*>[\s\n]*(<a[^>]*>)?[\s\n]*(https?://[^<\s\n]+)[\s\n]*(</a>)?[\s\n]*</p>)}m.freeze
 
       class << self
         # Find URLs on paragraphs.  We do it after rendering because
@@ -13,7 +13,7 @@ module Jekyll
         # catch both.
         def embed!(content)
           content.gsub!(URL_RE) do |match|
-            url = CGI.unescapeHTML($3)
+            url = CGI.unescapeHTML(Regexp.last_match(3))
             embed = Jekyll::Embed.embed url
 
             if embed == url

data/lib/jekyll/embed.rb

@@ -13,7 +13,7 @@ require_relative 'embed/cache'
 if Gem::Version.new(Jekyll::VERSION) >= Gem::Version.new('4.2.0')
   require_relative 'embed/content'
 else
-  Jekyll.logger.warn "Upgrade to Jekyll >= 4.2.0 to embed URLs in content"
+  Jekyll.logger.warn 'Upgrade to Jekyll >= 4.2.0 to embed URLs in content'
 end
 
 OEmbed::Providers.register_all
@@ -57,7 +57,8 @@ module Jekyll
 
     # Directive from Feature Policy
     # @see {https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Feature-Policy#directives}
-    DIRECTIVES = %w[accelerometer ambient-light-sensor autoplay battery camera display-capture document-domain encrypted-media execution-while-not-rendered execution-while-out-of-viewport fullscreen gamepad geolocation gyroscope layout-animations legacy-image-formats magnetometer microphone midi navigation-override oversized-images payment picture-in-picture publickey-credentials-get speaker-selection sync-xhr usb screen-wake-lock web-share xr-spatial-tracking].freeze
+    DIRECTIVES = %w[accelerometer ambient-light-sensor autoplay battery camera display-capture document-domain
+                    encrypted-media execution-while-not-rendered execution-while-out-of-viewport fullscreen gamepad geolocation gyroscope layout-animations legacy-image-formats magnetometer microphone midi navigation-override oversized-images payment picture-in-picture publickey-credentials-get speaker-selection sync-xhr usb screen-wake-lock web-share xr-spatial-tracking].freeze
 
     # Templates
     INCLUDE_OGP = '{% include ogp.html site=site page=page %}'
@@ -84,13 +85,13 @@ module Jekyll
         'rel' => %w[noopener noreferrer],
         'target' => '_blank'
       }
-    }
+    }.freeze
 
     class << self
       def site
         unless @site
           raise Jekyll::Errors::InvalidConfigurationError,
-            "Site is missing, configure with `Jekyll::Embed.site = site`"
+                'Site is missing, configure with `Jekyll::Embed.site = site`'
         end
 
         @site
@@ -101,7 +102,7 @@ module Jekyll
       # @param [Jekyll::Site]
       # @return [Jekyll::Site]
       def site=(site)
-        raise ArgumentError, "Site must be a Jekyll::Site" unless site.is_a? Jekyll::Site
+        raise ArgumentError, 'Site must be a Jekyll::Site' unless site.is_a? Jekyll::Site
 
         @site = site
 
@@ -130,15 +131,15 @@ module Jekyll
       def reset
         @allow_same_origin =
           @cache =
-          @config =
-          @fallback_template =
-          @get_cache =
-          @http_client =
-          @info =
-          @ogp_template =
-          @payload =
-          @value_for_attr =
-          nil
+            @config =
+              @fallback_template =
+                @get_cache =
+                  @http_client =
+                    @info =
+                      @ogp_template =
+                        @payload =
+                          @value_for_attr =
+                            nil
       end
 
       # Render the URL as HTML
@@ -170,7 +171,9 @@ module Jekyll
       # @return [Hash]
       def config
         @config ||= Jekyll::Utils.deep_merge_hashes(DEFAULT_CONFIG, (site.config['embed'] || {})).tap do |c|
-          c['attributes']['allow'].concat (DIRECTIVES - c.dig('attributes', 'allow').join.split(';').map { |s| s.split(' ').first }).join(" 'none';|").split('|')
+          c['attributes']['allow'].concat (DIRECTIVES - c.dig('attributes', 'allow').join.split(';').map do |s|
+                                                          s.split(' ').first
+                                                        end).join(" 'none';|").split('|')
         end
       end
 
@@ -208,8 +211,8 @@ module Jekyll
 
           %w[image video audio].each do |attr|
             page[attr] = ogp.public_send(:"#{attr}s").find do |a|
-              http? i.url
-            end
+              http? a.url
+            end&.url
           end
 
           context = info.dup
@@ -343,8 +346,8 @@ module Jekyll
         @value_for_attr ||= {}
         @value_for_attr[attr] ||=
           case (value = config.dig('attributes', attr))
-            when String then value
-            when Array then value.join(' ')
+          when String then value
+          when Array then value.join(' ')
           end
       end
 
@@ -355,7 +358,7 @@ module Jekyll
       # @return [String]
       def allow_same_origin(url)
         unless site.config['url']
-          Jekyll.logger.warn "Add url to _config.yml to determine if the site can embed itself"
+          Jekyll.logger.warn 'Add url to _config.yml to determine if the site can embed itself'
           return ' allow-same-origin'
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-embed-urls
 version: !ruby/object:Gem::Version
-  version: 0.5.6
+  version: 0.5.7
 platform: ruby
 authors:
 - f
@@ -128,6 +128,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
+- !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'
 description: Replaces URLs for their previsualization in Jekyll posts
 email:
 - f@sutty.nl