Message d'alerte généré par le core tronqué dans le cas d'une alerte sur valeur inférieure ou égale

Spine · Mai 15, 2022, 4:08

Bonjour,

Je constate qu’il manque la fin du message d’alerte généré par le core lorsque l’on définit une alerte sur valeur avec l’opérateur de comparaison <= pour le niveau warning ainsi que pour le niveau danger. Par acquis de conscience j’ai testé les autres cas et on voit bien que c’est uniquement ce cas qui pose problème :
Les alertes sur valeur définies :

Et les messages générés par le core dans l’ordre inverse des alertes sur valeur définies ci-dessus :

Il manque donc l’opérateur de comparaison et la valeur définie dans l’alerte dans ce cas précis.

Merci d’avance.

Edit : testé en Alpha 4.3.0 : même constat.

Edit2 : après analyse c’est la fonction strip_tags de la méthode secureXSS appelée par la méthode add de la classe message qui considère <= comme une balise et tronque le message :

github.com

jeedom/core/blob/dd456df47821c0360c9c1f00bb94ffe88885ac5c/core/php/utils.inc.php#L1034


      
          
          /**
           *
           * @param string $_string
           * @return string
           */
          function secureXSS($_string) {
          	if ($_string === null) {
          		return null;
          	}
          	return str_replace('&amp;', '&', htmlspecialchars(strip_tags($_string), ENT_QUOTES | ENT_HTML5, 'UTF-8'));
          }
          
          function minify($_buffer) {
          	$search = array(
          		'/\>[^\S ]+/s', // strip whitespaces after tags, except space
          		'/[^\S ]+\</s', // strip whitespaces before tags, except space
          		'/(\s)+/s', // shorten multiple whitespace sequences
          	);
          	$replace = array(
          		'>',

Jeedom : 4.2.15
Debian : 10.12

Loic · Juin 30, 2022, 1:36

Bonjour,
Merci pour le retour malheureusement la je vois pas comment corriger sans ouvrir une faille de securité sur vos jeedoms. Je preferes donc laisser comme ca.

Spine · Juin 30, 2022, 1:46

Bonjour,

Merci pour votre retour, je comprends votre position et je vais donc mettre en place une solution de contournement.

system · Juillet 1, 2022, 1:46

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.