Update allowed_protocols config for compatibility with Dompdf v3.1 (#…

…1071)
barryvdh · Jan 21, 2025 · 8d698b4 · 8d698b4
1 parent d2b3a15
commit 8d698b4
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/config/dompdf.php b/config/dompdf.php
@@ -91,6 +91,7 @@
          * @var array
          */
         'allowed_protocols' => [
+            'data://' => ['rules' => []],
             'file://' => ['rules' => []],
             'http://' => ['rules' => []],
             'https://' => ['rules' => []],

diff --git a/tests/PdfTest.php b/tests/PdfTest.php
@@ -163,4 +163,14 @@ public function testMultipleInstances(): void
         $this->assertEquals('host1', $pdf1->getDomPDF()->getBaseHost());
         $this->assertEquals('host2', $pdf2->getDomPDF()->getBaseHost());
     }
+
+    public function testDataImage(): void
+    {
+        $pdf = Facade\Pdf::loadHTML('<img src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAoAAAAKCAIAAAACUFjqAAAAEklEQVR4nGP8z4APMOGVHbHSAEEsAROxCnMTAAAAAElFTkSuQmCC" />');
+        $response = $pdf->download('test.pdf');
+
+        $this->assertInstanceOf(Response::class, $response);
+        $this->assertNotEmpty($response->getContent());
+        $this->assertEquals(1424, strlen($response->getContent()));
+    }
 }