jekyll-embed-urls 0.4.5 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec91f328eea6a0559fa41ffbfec9bccd3e3a19662369c36331cfaa8d6dc8d4fc
-  data.tar.gz: 47b8560d986e6083de1538ab0a372756705579af46d8b4d725051f2f52ad15f9
+  metadata.gz: 3c93e9e9f911c9988829cddbd88afd919995cfd55fd51a21e917e81f7044ddbf
+  data.tar.gz: 748730c8fb90b9f37f02b5cb7461a08df1502a1700a02748837450451ab0627a
 SHA512:
-  metadata.gz: 5cb5c650b7e79ec5ff0ea074acdbf2334dce4f310a1cc670613f31313dce81368a8a216d468bd73762fab08a13ccfe34d5f5d6887602d8fcc80f07dba5e8c120
-  data.tar.gz: f3b16a6d48a274cdd83fa710d5e0e7cec88bd281bd578dba5dd7bccba3d509f9d3a7ecb124f058c73e11687ea273d56c4194599ad552cdf35e4aac3c1b033420
+  metadata.gz: 2ac2e6bd99303c206cbcbe865d91f39cfc553c3d5f02d36c1ec88760a2fc757b805235f6054ae7f6d7d694d69e4f94af18a4202f635717ae13541cb9e491786c
+  data.tar.gz: f85276cd1f18db800686ff00f1cd00a83535c0c16a968b31175e6c4ba8f02492f791fdfd714de7657ac175d78da023f23a716bde46da55497ec2ac6d4a107897

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

@@ -157,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)
@@ -178,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
@@ -196,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
@@ -209,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
@@ -223,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
@@ -252,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|
@@ -291,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)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-embed-urls
 version: !ruby/object:Gem::Version
-  version: 0.4.5
+  version: 0.5.0
 platform: ruby
 authors:
 - f
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-24 00:00:00.000000000 Z
+date: 2022-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll