Bug cmd.class.php

m.georgein · Juillet 2, 2025, 3:01

bonjour,

Depouis qq temps j’avais des soucis de volume dsk qui augmenter sans cesse de 1 go ou plus par jour, apres investigation j’ai trouvé le coupable les log apaches, 1.5 go de log , des millions de lignes, avec la même erreur portant sur la ligne 1674 de cmd.class.php.

En fait il manque un contrôle

if ($cmdValue->execCmd() == $coupleArray[0] || $cmdValue->execCmd() == $coupleArray[1]) {

Si le premier segment n’existe pas, cela provoque une erreur apparemment sans incidences autres que celle de gonfler le log.

en mettant

if ($cmdValue->execCmd() == $coupleArray[0] || (isset($coupleArray[1]) && $cmdValue->execCmd() == $coupleArray[1])) {
    if (isset($coupleArray[1])) {

tout rentre dans l’ordre

A remarquer que qq lignes plus loin on a bien

} else { if (isset($coupleArray[1])) {

La, le contrôle est bien en place

Depuis les logs d’apache sont tout calme et jeedom respire nettement mieux malgré la chaleur !!!

Tonio16 · Juillet 2, 2025, 4:49

Pourqou ne pas proposer la modification sur le github ?

github.com

jeedom/core/blob/a9ed6907b4bf94c5f890eca5512a090f919e9498/core/class/cmd.class.php#L1674


      
          			'#value_history#' => ''
          		);
          		if ($this->getConfiguration('listValue', '') != '') {
          			$listOption = '';
          			$elements = explode(';', $this->getConfiguration('listValue', ''));
          			$foundSelect = false;
          			foreach ($elements as $element) {
          				$coupleArray = explode('|', $element);
          				$cmdValue = $this->getCmdValue();
          				if (is_object($cmdValue) && $cmdValue->getType() == 'info') {
          					if ($cmdValue->execCmd() == $coupleArray[0] || $cmdValue->execCmd() == $coupleArray[1]) {
          						$listOption .= '<option value="' . $coupleArray[0] . '" selected>' . $coupleArray[1] . '</option>';
          						$foundSelect = true;
          					} else {
          						$listOption .= '<option value="' . $coupleArray[0] . '">' . $coupleArray[1] . '</option>';
          					}
          				} else {
          					if (isset($coupleArray[1])) {
          						$listOption .= '<option value="' . $coupleArray[0] . '">' . $coupleArray[1] . '</option>';
          					} else {
          						$listOption .= '<option value="' . $coupleArray[0] . '">' . $coupleArray[0] . '</option>';

jpty · Juillet 2, 2025, 5:27

Vous devez avoir une cmd de type action sous-type liste pour laquelle vous n’avez pas respecté la spécification lors de la construction de la liste:

val1|texte1;val2|texte2 ....

Phpvarious · Juillet 2, 2025, 5:43

Bonsoir,

Il y a déja un PR en attente sur cette partie, je sais pas si celui-ci résout aussi ce cas.

github.com/jeedom/core

Fix command select using key OR value for selected

jeedom:alpha ← Flobul:alpha

opened 06:50AM - 20 Apr 25 UTC

Flobul

+8 -1

Fix command select on widget using key AND value for selected value. Give the c…hoice to use key or value from #listValue#  ## Description  When #listValue# has same kind of value for key/value (i.e.` 1|7;2|6;3|5;4|4;5|3;6|2;7|1`), using the actuel condition broke the selected value. ### Suggested changelog entry Add option in command configuration to choose between key and value for the selected listValue ### Related issues/external references Fixes # Fix bug introduced here https://github.com/jeedom/core/pull/2675 ## Types of changes - [X] Bug fix _(non-breaking change which fixes)_ - [X] New feature _(non-breaking change which adds functionality)_ - [ ] Breaking change _(fix or feature that would cause existing functionality to change)_ - [ ] This change is only breaking for integrators, not for external standards or end-users. - [ ] Documentation improvement ## PR checklist - [x] I have checked there is no other PR open for the same change. - [x] I have read the [[La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project](.github/CONTRIBUTING.md)). - [x] I grant the project the right to include and distribute the code under the GNU. - [x] I have added tests to cover my changes. - [x] I have verified that the code complies with the projects coding standards. - [ ] [Required for new sniffs] I have added MD documentation for the sniff.

Même avis que jpty, tu dois avoir une liste mal configurée.

m.georgein · Juillet 2, 2025, 11:09

Pas trop envie de me plonger dans les subtilités du github pour une petite modif

m.georgein · Juillet 2, 2025, 11:29

Autant chercher une aiguille dans une botte de foin

Tonio16 · Juillet 3, 2025, 3:42

Oui mais c’est la méthode la plus sur en fait.

Antoine

jpty · Juillet 3, 2025, 8:09

Bonjour,
Vous avez trouvé ce qu’il fallait corriger dans le source. Il suffit de mettre un message dans le code pour avoir l’ID de la commande …

		if ($this->getConfiguration('listValue', '') != '') {
			$listOption = '';
			$elements = explode(';', $this->getConfiguration('listValue', ''));
			$foundSelect = false;
			foreach ($elements as $element) {
				$coupleArray = explode('|', $element);
if(count($coupleArray) != 2) 
  message::add(__CLASS__, "EqLogic: [" .$this->getEqLogic()->getName() ."] CmdId: " .$this->getId() ." Erreur in listValue:[" .$element ."]");
				$cmdValue = $this->getCmdValue();
				if (is_object($cmdValue) && $cmdValue->getType() == 'info') {
					if ($cmdValue->execCmd() == $coupleArray[0] || $cmdValue->execCmd() == $coupleArray[1]) {
						$listOption .= '<option value="' . $coupleArray[0] . '" selected>' . $coupleArray[1] . '</option>';

m.georgein · Juillet 3, 2025, 11:33

!!! Mais bon sang ! Bien sur !

avant

0|Off;1|On;2Auto

après

0|Off;1|On;2|Auto

Merci pour le coup de projecteur !!!, c’est une ligne a laissé dans la class je pense !!!

jpty · Juillet 3, 2025, 12:18

J’ai du mal à comprendre comment ça pouvait générer 1Go de log par jour.
Il y en avait d’autre ?

m.georgein · Juillet 3, 2025, 1:39

Non, seulement celui-là, j’ai laissé le message actif dans cmd.class au cas où il y en aurait d’autres, c’est vrai que plus de 1 go/jour c’est énorme pour un seul virtuel

Je viens de retrouver un des tests que j’avais fait avec un échantillon de log apache :

[Wed Jul 02 15:52:05.596133 2025] [php:warn] [pid 811461:tid 811461] [client 192.168.2.241:60028] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:52:05.596143 2025] [php:warn] [pid 811461:tid 811461] [client 192.168.2.241:60028] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:52:05.602067 2025] [php:warn] [pid 810356:tid 810356] [client 192.168.2.241:60014] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:52:05.602079 2025] [php:warn] [pid 810356:tid 810356] [client 192.168.2.241:60014] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:52:06.723359 2025] [php:warn] [pid 730652:tid 730652] [client 192.168.2.241:60026] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:52:06.723375 2025] [php:warn] [pid 730652:tid 730652] [client 192.168.2.241:60026] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:52:36.826770 2025] [php:warn] [pid 744903:tid 744903] [client 192.168.2.241:60031] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:52:36.826787 2025] [php:warn] [pid 744903:tid 744903] [client 192.168.2.241:60031] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:52:38.908641 2025] [php:warn] [pid 736822:tid 736822] [client 192.168.2.241:60032] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:52:38.908654 2025] [php:warn] [pid 736822:tid 736822] [client 192.168.2.241:60032] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:53:03.084907 2025] [php:warn] [pid 814911:tid 814911] [client 192.168.2.241:60050] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:53:03.084925 2025] [php:warn] [pid 814911:tid 814911] [client 192.168.2.241:60050] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:53:07.751698 2025] [php:warn] [pid 783340:tid 783340] [client 192.168.2.241:60066] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:53:07.751711 2025] [php:warn] [pid 783340:tid 783340] [client 192.168.2.241:60066] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:53:07.752547 2025] [php:warn] [pid 706171:tid 706171] [client 192.168.2.241:60061] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:53:07.752559 2025] [php:warn] [pid 706171:tid 706171] [client 192.168.2.241:60061] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:53:07.753948 2025] [php:warn] [pid 736822:tid 736822] [client 192.168.2.241:60060] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:53:07.753960 2025] [php:warn] [pid 736822:tid 736822] [client 192.168.2.241:60060] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675
[Wed Jul 02 15:53:07.771855 2025] [php:warn] [pid 756711:tid 756711] [client 192.168.2.241:60062] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1674
[Wed Jul 02 15:53:07.771868 2025] [php:warn] [pid 756711:tid 756711] [client 192.168.2.241:60062] PHP Warning:  Undefined array key 1 in /var/www/html/core/class/cmd.class.php on line 1675

On a une fréquence de répétition énorme, apparemment plus de 50 x sec avec 2 lgn à chaque fois ce qui, sauf erreur de ma part, pour 190 octets la ligne donne en 24 h :
2 * 190 * 3600 * 24 * 50 => 1 641 600 000 => 1.6 go

Perso, je ne vois pas du tout ce qui peut provoquer un tel niveau de répétition

Aurel · Juillet 3, 2025, 1:57

Tu n’a pas une boucle quelque part ?
Genre un virtuel qui appelle un virtuel à chaque maj ?

jpty · Juillet 3, 2025, 2:04

Avec 1.6Go de log généré par jour, ça vaudrait le coup qu’une correction soit faite dans le core pour une diffusion avant la version 5 de Jeedom.

Vous devez avoir aussi un petit souci avec les logs. Chez moi en 4.4(Deb11) ou en 4.5(Deb12), les php notice,warning,error arrivent dans le log Jeedom http.error. Les logs apache sont quasiment vides.
C’est peut-être les 50 logs à la seconde qui font que les logs restent au niveau apache.