analysis: do not parse/check empty HTML (it produced unnecessary warning) - it is valid to have content-type: text/html but with connect-lengt: 0 (for example case for 'gtm.js?id=')

Author: janreges

src/Crawler/Analysis/Manager.php

@@ -140,7 +140,7 @@ public function analyzeVisitedUrl(VisitedUrl $visitedUrl, ?string $body, ?array
         $result = [];
 
         $dom = null;
-        if ($visitedUrl->contentType === Crawler::CONTENT_TYPE_ID_HTML) {
+        if ($visitedUrl->contentType === Crawler::CONTENT_TYPE_ID_HTML && $body !== null && trim($body) !== '') {
             $s = microtime(true);
             $encodedBody = mb_convert_encoding($body, 'HTML-ENTITIES', 'UTF-8');
             if (!$encodedBody) {

src/Crawler/Analysis/SecurityAnalyzer.php

@@ -128,7 +128,10 @@ public function analyzeVisitedUrl(VisitedUrl $visitedUrl, ?string $body, ?DOMDoc
         $result = new UrlAnalysisResult();
 
         $this->checkHeaders($headers, $visitedUrl->isHttps(), $result);
-        $this->checkHtmlSecurity($body, $visitedUrl->isHttps(), $result);
+
+        if ($body !== null && trim($body) !== '') {
+            $this->checkHtmlSecurity($body, $visitedUrl->isHttps(), $result);
+        }
 
         return $result;
     }