From 17afb469945d2cfd9c6158341e393e19bc4a1ffb Mon Sep 17 00:00:00 2001
From: Ivan Paramonau <i.paramonau@gmail.com>
Date: Wed, 28 Aug 2024 11:05:23 -0400
Subject: [PATCH] domains: fix canonical URL for Facebook's shared videos

---
 plugins/domains/facebook.com/facebook.thumbnail.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/plugins/domains/facebook.com/facebook.thumbnail.js b/plugins/domains/facebook.com/facebook.thumbnail.js
index 051acf814..d97c99f87 100644
--- a/plugins/domains/facebook.com/facebook.thumbnail.js
+++ b/plugins/domains/facebook.com/facebook.thumbnail.js
@@ -54,12 +54,17 @@ export default {
     },
 
 
-    getData: function(oembed, options) {
+    getData: function(oembed, url, options) {
         
         if (oembed.html && /class=\"fb\-(post|video)\"/i.test(oembed.html) 
             && options.getProviderOptions('facebook.thumbnail', true) && !/comment_id=/.test(oembed.html)) {
 
-            options.followHTTPRedirect = true; // avoid security re-directs of URLs if any
+            // Avoid security re-directs of URLs if any,
+            // but fix canonical URLs for /share urls that now redirect to story.php
+            if (!/facebook\.com\/(permalink|story)\.php\?/i.test(url)) {
+                options.followHTTPRedirect = true; 
+            }
+
             options.exposeStatusCode = true;
             options.provider = 'Facebook';