6 appels de la fonction system::getCmdSudo()

Salon des Développeurs Core V4 (dev)
1

Bonjour,

6 appels de la fonction system::getCmdSudo()
Qui peut croire que le résultat de ces appels va varier ?

Les sources des fonctions appelées:

     public static function getCmdSudo(): string {
		if (!class_exists('jeedom')) {
			return 'sudo ';
		}
		if (!jeedom::isCapable('sudo')) {
			return '';
		}
		return 'sudo ';
	}

	public static function isCapable($_function, $_forceRefresh = false) {
		global $JEEDOM_COMPATIBILIY_CONFIG;
		if ($_function == 'sudo') {
			if (!$_forceRefresh) {
				$cache = cache::byKey('jeedom::isCapable::sudo');
				if ($cache->getValue(0) == 1) {
					return true;
				}
			}
			$result = (shell_exec('sudo -l > /dev/null 2>&1; echo $?') == 0) ? true : false;
			cache::set('jeedom::isCapable::sudo', $result, 3600 * 24);
			return $result;
		}
		$hardware = self::getHardwareName();
		if (!isset($JEEDOM_COMPATIBILIY_CONFIG[$hardware])) {
			return false;
		}
		if (in_array($_function, $JEEDOM_COMPATIBILIY_CONFIG[$hardware])) {
			return true;
		}
		return false;
	}

Je n’ai pas mis la fonction getHardwareName et quelques autres.

2

Salut,

Je me suis fait la même réflexion suite à notre sujet sur les problèmes de vidage des logs. C’est vrai que c’est surprenant de pas avoir crée une variable dans ce cas de figure :face_with_raised_eyebrow:

3

Problème de vidage des logs toujours en réflexion : Jeedom 4.5 - un manque de logs dans les scénarios - #26 par Loic
Pas de réaction à ma proposition de mettre le fichier intermédiaire en tmpfs

4

Hello,

Et moi je ne comprends pas le besoin d’une fonction qui test si sudo est installé, vu qu’il l’est forcement lors de la step 2 de l’intallation du Core :

Bad

5

C’est peut-être une tentative de réduction du nombre d’appel de sudo qui la plupart du temps est inutile. En fonctionnement normal, les fichiers appartiennent à www-data:www-data et ne nécessite pas un traitement par sudo.

1 « J'aime »
6

Je propose un autre sujet pour ce débat histoire de ne pas polluer le thread de @scanab …

1 « J'aime »
7

Sudo peut ne pas être la. Il est recommandé mais dans certain cas d’installation critique au niveau securité l’installateur préfère ne pas mettre sudo sur le serveur et ne pas donner les droits root a jeedom d’où ce test.

8

Cest toujours en cours de réflexion le tempfs est une bonne idée mais vu qu’il est de petite taille il faut que je sois sûr de pas avoir des fichiers de logs trop gros. En plus ya toujours un soucis quand c’est le fichier des logs des démon ou en faisant un replace du fichier de log on peut se retrouver dans des cas étrange ou le fichier est toujours là et grossi jusqu’au redémarrage du démon.

1 « J'aime »
9

Salut,

Merci pour l’explication sur la possible non présence de la commande sudo. Par contre lors d’un petit travail d’optimisation, ça vaudrait quand même le coup de mettre le résultat dans une variable pour ne pas rappeler la fonction system::getCmdSudo(), non ?

10

C’est fait dans isCapable :

Mais il est effectivement aussi possible de mettre "sudo " dans le cache.

11

Vu le nombre d’endroits où sudo est référencé directement, je ne suis pas sur que Jeedom soit fonctionnel sans, mais je te crois sur parole.

12

Je dis pas que tout marche mais ça convient pour le moment aux personnes qui l’utilisent sans sudo.