Produits Entr'ouvert » Hobo

Déjà l’approche de base à chercher dans le code pour la chaîne '<script' n’est plus suffisante à partir du moment où on remplace du contenu.
Je pense que la bonne approche c’est de tenter de parser le contenu comme du html, et sur un succès seulement et présence d’une balise script détectée au parsage, alors on remplace le contenu global par le contenu de la balise script. On laisse le contenu inchangé sinon.

Dans ma jeunesse, parser du HTML en XML, c'était le plantage assuré, les temps changent :)

Mais sinon le code me semble un peu compliqué, j'imaginais plutôt le parsing XML complétement contenu dans un try/except, genre :

        try:
            xml_node = etree.fromstring(value)
            if xml_node.tag == 'script':
                script_tag = xml_node
            else:
                script_tag = xml_node.find('script')
            if script_tag is not None:
                return script_tag.text
        except etree.ParseError:
            pass

J'aurais du mentionner l'HTML proposé par Matomo, qu'il soit utilisé dans les tests plutôt que le fictif <script>...js_code_3...</script>.sur une réalité :

<!-- Matomo -->
<script type="text/javascript">
  var _paq = window._paq = window._paq || [];
  /* tracker methods like "setCustomDimension" should be called before "trackPageView" */
  _paq.push(['trackPageView']);
  _paq.push(['enableLinkTracking']);
  (function() {
    var u="https://.../";
    _paq.push(['setTrackerUrl', u+'matomo.php']);
    _paq.push(['setSiteId', '112']);
    var d=document, g=d.createElement('script'), s=d.getElementsByTagName('script')[0];
    g.type='text/javascript'; g.async=true; g.src=u+'matomo.js'; s.parentNode.insertBefore(g,s);
  })();
</script>
<!-- End Matomo Code -->

Automatic expiration