Projet

Général

Profil

Development #32948

ajouter un message d'erreur quand de l'html est ajouté au code de suivi js

Ajouté par Frédéric Péters il y a presque 5 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Lauréline Guérin
Catégorie:
-
Version cible:
-
Début:
09 mai 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Genre détecter <script et lever ValidationError(...).

Fichiers

0001-matomo-detect-html-tags-in-tracking_js-32948.patch (3,72 ko) 0001-matomo-detect-html-tags-in-tracking_js-32948.patch Lauréline Guérin, 04 février 2020 15:26
0001-matomo-detect-html-tags-in-tracking_js-32948.patch (4,05 ko) 0001-matomo-detect-html-tags-in-tracking_js-32948.patch Lauréline Guérin, 10 février 2020 10:19
0001-matomo-detect-html-tags-in-tracking_js-32948.patch (4,03 ko) 0001-matomo-detect-html-tags-in-tracking_js-32948.patch Lauréline Guérin, 10 février 2020 10:21
0001-matomo-detect-html-tags-in-tracking_js-32948.patch (4,29 ko) 0001-matomo-detect-html-tags-in-tracking_js-32948.patch Lauréline Guérin, 10 février 2020 10:36

Révisions associées

Révision 0ed638b8 (diff)
Ajouté par Lauréline Guérin il y a environ 4 ans

matomo: detect html tags in tracking_js (#32948)

Historique

#1

Mis à jour par Lauréline Guérin il y a environ 4 ans

  • Assigné à mis à Lauréline Guérin
#2

Mis à jour par Lauréline Guérin il y a environ 4 ans

#3

Mis à jour par Nicolas Roche il y a environ 4 ans

Étrangement, j'ai l'impression que l'expression xpath './/*' (bien que correcte) ne semble pas fonctionner correctement.
(cf http://effbot.org/zone/element-xpath.htm)
Elle détecte bien '<script>' mais pas les autres balises comme '<h1>' par exemple.
Sinon, il faut prévoir une condition pour ne pas passer un document vide au parseur html (ou récupérer l'exception).

#4

Mis à jour par Nicolas Roche il y a environ 4 ans

En fait, je pense qu'un parseur HTML ne permet pas détecter des balises HTML contenues dans un text,
comme par exemple "coucou<h1>", si j'ai bien compris le ticket.
  • lxml ajoute des <p> pour convertir le texte passé en HTML.
    > lxml.etree.tostring(lxml.html.fromstring('coucou<h1>'))
'<div><p>coucou</p><h1/></div>'
  • elementTree refuse de le parser.
    > ET.fromstring('coucou<h1>')
*** ParseError: not well-formed
#5

Mis à jour par Frédéric Péters il y a environ 4 ans

Faut pas se prendre la tête ici, le seul objectif c'est de signaler à l'usager qu'il a copié/collé <script... parce que c'est le bout que propose Matomo ou Google Analytics. Un if '<script' in blah serait même suffisant.

#6

Mis à jour par Lauréline Guérin il y a environ 4 ans

ok, je vais simplifier :)

#9

Mis à jour par Frédéric Péters il y a environ 4 ans

Marquer pour traduction, et côté texte je proposerais "This field should only contain the Javascript code. You should remove the surrounding <script> markup."

#11

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée
#12

Mis à jour par Lauréline Guérin il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0ed638b83e2ae5475f142e3c2554084479b03ec0
Date:   Tue Feb 4 15:23:36 2020 +0100

    matomo: detect html tags in tracking_js (#32948)
#13

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF