Restore 2 Disabled Tests

Author: oleibman

src/PhpSpreadsheet/Reader/Xlsx.php

@@ -1416,7 +1416,7 @@ protected function loadSpreadsheetFromFile(string $filename): Spreadsheet
                                                         );
                                                         if (isset($images[$linkImageKey])) {
                                                             $url = str_replace('xl/drawings/', '', $images[$linkImageKey]);
-                                                            $objDrawing->setPath($url);
+                                                            $objDrawing->setPath($url, false);
                                                         }
                                                         if ($objDrawing->getPath() === '') {
                                                             continue;
@@ -1509,7 +1509,7 @@ protected function loadSpreadsheetFromFile(string $filename): Spreadsheet
                                                         );
                                                         if (isset($images[$linkImageKey])) {
                                                             $url = str_replace('xl/drawings/', '', $images[$linkImageKey]);
-                                                            $objDrawing->setPath($url);
+                                                            $objDrawing->setPath($url, false);
                                                         }
                                                         if ($objDrawing->getPath() === '') {
                                                             continue;

src/PhpSpreadsheet/Worksheet/Drawing.php

@@ -109,7 +109,12 @@ public function setPath(string $path, bool $verifyFile = true, ?ZipArchive $zip
             }
             // Implicit that it is a URL, rather store info than running check above on value in other places.
             $this->isUrl = true;
-            $imageContents = @file_get_contents($path);
+            $ctx = null;
+            // https://github.com/php/php-src/issues/16023
+            if (str_starts_with($path, 'https:')) {
+                $ctx = stream_context_create(['ssl' => ['crypto_method' => STREAM_CRYPTO_METHOD_TLSv1_3_CLIENT]]);
+            }
+            $imageContents = @file_get_contents($path, false, $ctx);
             if ($imageContents !== false) {
                 $filePath = tempnam(sys_get_temp_dir(), 'Drawing');
                 if ($filePath) {

tests/PhpSpreadsheetTests/Reader/Xlsx/URLImageTest.php

@@ -12,8 +12,7 @@
 
 class URLImageTest extends TestCase
 {
-    // https://github.com/readthedocs/readthedocs.org/issues/11615
-    public function xtestURLImageSource(): void
+    public function testURLImageSource(): void
     {
         if (getenv('SKIP_URL_IMAGE_TEST') === '1') {
             self::markTestSkipped('Skipped due to setting of environment variable');
@@ -41,6 +40,10 @@ public function xtestURLImageSource(): void
             self::assertNotFalse($mimeType);
             $extension = File::mime2ext($mimeType);
             self::assertSame('png', $extension);
+            self::assertSame(IMAGETYPE_PNG, $drawing->getType());
+            self::assertSame(84, $drawing->getWidth());
+            self::assertSame(44, $drawing->getHeight());
+            $spreadsheet->disconnectWorksheets();
         }
     }
 
@@ -56,6 +59,7 @@ public function xtestURLImageSourceNotFound(): void
         $worksheet = $spreadsheet->getActiveSheet();
         $collection = $worksheet->getDrawingCollection();
         self::assertCount(0, $collection);
+        $spreadsheet->disconnectWorksheets();
     }
 
     public function testURLImageSourceBadProtocol(): void