Merge pull request #101 from j0k3r/ng-controller

ng-controller should not trigger escaped fragment
j0k3r · May 11, 2017 · a8501be · a8501be
2 parents 3db9a94 + 15a530e
commit a8501be
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/src/Extractor/HttpClient.php b/src/Extractor/HttpClient.php
@@ -60,7 +60,6 @@ public function __construct(Client $client, $config = [], LoggerInterface $logge
                 '<meta name="fragment" content="!"',
                 "<meta content='!' name='fragment'",
                 '<meta content="!" name="fragment"',
-                ' ng-controller=',
             ],
             // timeout of the request in seconds
             'timeout' => 10,

diff --git a/tests/Extractor/HttpClientTest.php b/tests/Extractor/HttpClientTest.php
@@ -302,11 +302,6 @@ public function dataForMetaRefresh()
                 '<html><meta name="fragment" content="!"></html>',
                 'http://www.bernama.com/bernama/v6/newsindex.php?id=943513',
             ],
-            [
-                'http://fr.wikipedia.org/wiki/Copyright',
-                '<html><body ng-controller="MyCtrl"></body></html>',
-                'http://www.bernama.com/bernama/v6/newsindex.php?id=943513',
-            ],
         ];
     }