From: Antonio Ospite <ao2@ao2.it>
Date: Fri, 7 Jan 2022 20:44:51 +0000 (+0100)
Subject: Revert "src/Tweeper.php: fix rendering Instagram images in some feed readers"
X-Git-Url: https://git.ao2.it/tweeper.git/commitdiff_plain/2c7f7e5da9efb2905d9c87cb5a9a2d6c057f576c?ds=inline;hp=6525c19868a0511abaaac9d2ba452ba640899209

Revert "src/Tweeper.php: fix rendering Instagram images in some feed readers"

This reverts commit 6525c19868a0511abaaac9d2ba452ba640899209.

The problem was not really about Instagram using Cross Origin Resource
Policy but probably more about Liferea not parsing the images URLs
correctly.

So revert the change, since this broke images in other feed readers
which do not support Data URLs, like for example newsboat.
---

diff --git a/src/Tweeper.php b/src/Tweeper.php
index 2d3870f..f79bb08 100644
--- a/src/Tweeper.php
+++ b/src/Tweeper.php
@@ -234,34 +234,6 @@ class Tweeper {
   }
 
   /**
-   * Generate a data URL.
-   */
-  public static function generateDataURL($url) {
-    $supported_content_types = [
-      "image/gif",
-      "image/jpeg",
-      "image/png",
-    ];
-
-    $url_info = Tweeper::getUrlInfo($url);
-    if (FALSE === $url_info) {
-      trigger_error("Failed to retrieve info for URL: " . $url, E_USER_WARNING);
-      return '';
-    }
-
-    $supported = in_array($url_info['content_type'], $supported_content_types);
-    if (!$supported) {
-      trigger_error("Unsupported data URL type \"" . $url_info['content_type'] . "\" for URL: " . $url_info['url'], E_USER_WARNING);
-      return '';
-    }
-
-    $base64Data = base64_encode(file_get_contents($url));
-    $dataURL = 'data: ' . $url_info['content_type'] . ';base64,' . $base64Data;
-
-    return $dataURL;
-  }
-
-  /**
    * Mimic the message from libxml.c::php_libxml_ctx_error_level()
    */
   private static function logXmlError($error) {
diff --git a/src/rss_converter_instagram.com.xsl b/src/rss_converter_instagram.com.xsl
index 71b60a7..855ce0b 100644
--- a/src/rss_converter_instagram.com.xsl
+++ b/src/rss_converter_instagram.com.xsl
@@ -66,7 +66,6 @@
 
     <xsl:template match="//edges/node">
         <xsl:variable name="item-content-image" select="./display_url"/>
-        <xsl:variable name="item-content-image-data" select="php:functionString('Tweeper\Tweeper::generateDataURL', $item-content-image)"/>
         <xsl:variable name="item-content-caption" select="./edge_media_to_caption/edges/node/text"/>
         <xsl:variable name="item-permalink" select="concat($BaseURL, '/p/', ./shortcode, '/')"/>
         <item>
@@ -109,7 +108,7 @@
                     <xsl:value-of select="$item-content-caption"/>
                 </p><br />
                 <xsl:if test="$show-multimedia = 1">
-                    <a href="{$item-permalink}"><img src="{$item-content-image-data}" style="max-width: 100%"/></a>
+                    <a href="{$item-permalink}"><img src="{$item-content-image}" style="max-width: 100%"/></a>
                 </xsl:if>
                 <xsl:text disable-output-escaping="yes">]]&gt;</xsl:text>
             </description>