Bladeren bron

Merge pull request #1505 from Alkarex/protocol-relative-favicon

Better handling of protocol-relative favicons
Alexandre Alapetite 9 jaren geleden
bovenliggende
commit
251f11c1d7
1 gewijzigde bestanden met toevoegingen van 6 en 1 verwijderingen
  1. 6 1
      lib/favicons.php

+ 6 - 1
lib/favicons.php

@@ -61,7 +61,12 @@ function searchFavicon(&$url) {
 					strtolower(trim($link->getAttribute('rel'))) === $rel) {
 					$href = trim($link->getAttribute('href'));
 					if (substr($href, 0, 2) === '//') {
-						$href = 'https:' . $href;
+						// Case of protocol-relative URLs
+						if (preg_match('%^(https?:)//%i', $url, $matches)) {
+							$href = $matches[1] . $href;
+						} else {
+							$href = 'https:' . $href;
+						}
 					}
 					if (filter_var($href, FILTER_VALIDATE_URL) === false) {
 						$href = SimplePie_IRI::absolutize($url, $href);