jekyll-embed-urls 0.4.4 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a414c302d33fdb35c35e420ae07e6fc5c8f90e2fba9b4ba2e8b2961fbd94a8f4
-  data.tar.gz: bf0d28a49c111e48d7f02d6ddc9fc258356b0e1dc3409870bb75206549cab968
+  metadata.gz: 214ec59b13ace086962b66c686ed56739a914773df1953f863fadaae1cc637b1
+  data.tar.gz: 7ad21fce1c1e5b6bedb59acb74c8454f6c25a9b56edb0c0009d63430e440bc6e
 SHA512:
-  metadata.gz: 9df2638f34fcbcee2a9ace50686fff8e22dff2c7168ca60673634032a97cf65fdb4b1e902e517aa81e5c1ffdc95e8a5f8eb4c53e3d160f6f872615b6b94cb7e6
-  data.tar.gz: eb64bf3fc323c3ec0159de2314aa8165c90a79ed95a21ecf497d8ec7aa2b79f4db3cf4e11c8386f0cf8a38c698e94af2c42b32440c656b044dbf216d780468b2
+  metadata.gz: 4cb5c6c3b3c68d64953548a5b15428427d3bffe5065382bbc00536386edf377aaa00154bb008009dca467dcca47e1c16b22e20ecc74af15648683965c3e2d770
+  data.tar.gz: cda3db036365ce8c06b942038fe9cd80da91fa330e63521209d1ebcc8c8a351299b5ae1a99df3670b3efdef01210944c4a7d8e12c7120f6d18b16af96a7bfb90

data/_includes/fallback.html

@@ -1,17 +1,17 @@
-<article class="fallback">
-  {%- if page.image -%}
-    <img referrerpolicy="{{ embed.referrerpolicy }}" loading="{{ embed.loading }}" src="{{ page.image }}" class="img-fluid" />
-  {%- endif -%}
+{% if page.url contains "//archive.org/" or page.url contains "//www.archive.org/" %}
+  <iframe src="{{ page.url | replace: "/details/", "/embed/" | uri_escape }}"></iframe>
+{% else %}
+  <article class="fallback" lang="{{ page.locale }}">
+    {%- if page.image -%}
+      <img src="{{ page.image | uri_escape }}" class="img-fluid" />
+    {%- endif -%}
 
-  <h1>{{ page.title }}</h1>
-  <p class="lead">{{ page.description }}</p>
-  <p><small>
-    <a
-      href="{{ page.url }}"
-      referrerpolicy="{{ embed.referrerpolicy }}"
-      target="{{ embed.target }}"
-      rel="{{ embed.rel }}">
-      {{ page.url }}
-    </a>
-  </small><p>
-</article>
+    <h1>{{ page.title }}</h1>
+    <p class="lead">{{ page.description }}</p>
+    <p>
+      <small>
+        <a href="{{ page.url | uri_escape }}">{{ page.url }}</a>
+      </small>
+    </p>
+  </article>
+{% endif %}

data/_includes/ogp.html

@@ -1,23 +1,19 @@
 <article class="ogp" lang="{{ page.locale }}">
   {%- if page.video -%}
-    <video poster="{{ page.image }}" class="img-fluid" {{ embed.controls }} src="{{ page.video }}"/>
+    <video poster="{{ page.image | uri_escape }}" class="img-fluid" src="{{ page.video | uri_escape }}"/>
   {%- elsif page.image -%}
-    <img referrerpolicy="{{ embed.referrerpolicy }}" loading="{{ embed.loading }}" src="{{ page.image }}" class="img-fluid" />
+    <img src="{{ page.image | uri_escape }}" class="img-fluid" />
   {%- endif -%}
 
   {%- if page.audio -%}
-    <audio class="img-fluid" {{ embed.controls }} src="{{ page.audio }}"/>
+    <audio class="img-fluid" src="{{ page.audio | uri_escape }}"/>
   {%- endif -%}
 
   <h1>{{ page.title }}</h1>
   <p class="lead">{{ page.description }}</p>
-  <p><small>
-    <a
-      href="{{ page.url }}"
-      referrerpolicy="{{ embed.referrerpolicy }}"
-      target="{{ embed.target }}"
-      rel="{{ embed.rel }}">
-      {{ page.url }}
-    </a>
-  </small><p>
+  <p>
+    <small>
+      <a href="{{ page.url | uri_escape }}">{{ page.url }}</a>
+    </small>
+  </p>
 </article>

data/lib/jekyll/embed.rb

@@ -107,10 +107,13 @@ module Jekyll
 
         # Add the _includes dir so we can provide default templates that
         # can be overriden locally or by the theme.
-        site.includes_load_paths << File.expand_path(File.join(__dir__, '..', '..', '_includes'))
+        includes_dir = File.expand_path(File.join(__dir__, '..', '..', '_includes'))
+        site.includes_load_paths << includes_dir unless site.includes_load_paths.include? includes_dir
         # Since we're embedding, we're allowing iframes
         Loofah::HTML5::SafeList::ALLOWED_ELEMENTS_WITH_LIBXML2 << 'iframe'
 
+        reset
+
         # Other elements that are disallowed
         config['scrub']&.each do |scrub|
           Loofah::HTML5::SafeList::ALLOWED_ELEMENTS_WITH_LIBXML2.delete(scrub)
@@ -121,6 +124,23 @@ module Jekyll
         site
       end
 
+      # Reset variables
+      #
+      # @return [nil]
+      def reset
+        @allow_same_origin =
+          @cache =
+          @config =
+          @fallback_template =
+          @get_cache =
+          @http_client =
+          @info =
+          @ogp_template =
+          @payload =
+          @value_for_attr =
+          nil
+      end
+
       # Render the URL as HTML
       #
       # 1. Try oembed for video and image
@@ -137,6 +157,7 @@ module Jekyll
         raise URI::Error unless url.start_with? 'http'
 
         # Just to verify the URL is valid
+        # TODO: Use Addressable
         URI.parse url
 
         oembed(url) || ogp(url) || fallback(url)
@@ -158,15 +179,13 @@ module Jekyll
       # @param [String] URL
       # @return [String,NilClass] Sanitized HTML or nil
       def oembed(url)
-        cache.getset(url) do
+        cache.getset("oembed+#{url}") do
           oembed = OEmbed::Providers.get url
 
           # Prevent caching of nil?
           raise OEmbed::Error unless oembed.respond_to? :html
 
-          # Cleanup.  We don't allow running remote scripts locally,
-          # period.
-          cleanup(Loofah.fragment(oembed.html).scrub!(:prune), url).to_s
+          cleanup oembed.html, url
         end
       rescue OEmbed::Error
         nil
@@ -176,12 +195,12 @@ module Jekyll
       # @param [String] URL
       # @return [String,NilClass]
       def ogp(url)
-        cache.getset(url) do
+        cache.getset("ogp+#{url}") do
           ogp = OGP::OpenGraph.new get(url).body
           context = info.dup
           context[:registers][:page] = payload['page'] = ogp.data
 
-          ogp_template.render! payload, context
+          cleanup ogp_template.render!(payload, context), url
         end
       rescue OGP::MalformedSourceError, OGP::MissingAttributeError, Faraday::Error
         nil
@@ -189,7 +208,7 @@ module Jekyll
 
       # Try something
       def fallback(url)
-        cache.getset(url) do
+        cache.getset("fallback+#{url}") do
           html        = Nokogiri::HTML.fragment get(url).body
           element     = html.css('article').first
           element   ||= html.css('section').first
@@ -203,10 +222,11 @@ module Jekyll
             'title' => text(title),
             'description' => text(description),
             'url' => url,
-            'image' => element&.css('img')&.first&.public_send(:[], 'src')
+            'image' => element&.css('img')&.first&.public_send(:[], 'src'),
+            'locale' => html.css('html')&.first&.public_send(:[], 'lang')
           }
 
-          fallback_template.render! payload, context
+          cleanup fallback_template.render!(payload, context), url
         end
       rescue Faraday::Error, Nokogiri::SyntaxError
         nil
@@ -232,7 +252,12 @@ module Jekyll
         end
       end
 
-      def cleanup(html, url)
+      # @params :html_fragment [String]
+      # @params :url [String]
+      # @return [String]
+      def cleanup(html_fragment, url)
+        html = Loofah.fragment(html_fragment).scrub!(:prune)
+
         # Add our own attributes
         html.css('iframe').each do |iframe|
           IFRAME_ATTRIBUTES.each do |attr|
@@ -271,8 +296,8 @@ module Jekyll
           element['href'] = CGI.escapeHTML(UrlPrivacy.clean(CGI.unescapeHTML(element['href'])))
         end
 
-        # Return the cleaned up HTML
-        html
+        # Return the cleaned up HTML as a String
+        html.to_s
       end
 
       def text(node)

data/lib/jekyll-embed-urls.rb

@@ -4,6 +4,6 @@ require_relative 'jekyll/embed'
 require_relative 'jekyll/embed/filter'
 
 # Configure Embed
-Jekyll::Hooks.register :site, :after_init do |site|
+Jekyll::Hooks.register :site, :after_reset do |site|
   Jekyll::Embed.site = site
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-embed-urls
 version: !ruby/object:Gem::Version
-  version: 0.4.4
+  version: 0.5.1
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-08 00:00:00.000000000 Z
+date: 2022-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -174,7 +174,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: Embed URL previsualization in Jekyll posts