From bb9868cafa6bc1be95bf2a1ae80b0bccc1dfc7fa Mon Sep 17 00:00:00 2001
From: jim-p <jimp@pfsense.org>
Date: Wed, 14 Aug 2013 11:24:16 -0400
Subject: For NRPE2, honor the "sudo" checkbox if it's checked and sudo exists
 on the system. Also, only add -w and -c if the boxes are filled in. Fixes
 #3145

---
 config/nrpe2/nrpe2.inc | 6 +++++-
 config/nrpe2/nrpe2.xml | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

(limited to 'config/nrpe2')

diff --git a/config/nrpe2/nrpe2.inc b/config/nrpe2/nrpe2.inc
index cd3fa013..25964b16 100644
--- a/config/nrpe2/nrpe2.inc
+++ b/config/nrpe2/nrpe2.inc
@@ -159,8 +159,12 @@ function nrpe2_custom_php_write_config() {
 	conf_mount_rw();
 	$cmds = array();
 	foreach ($config['installedpackages']['nrpe2']['config'][0]['row'] as $cmd) {
+		$sudo_bin = "/usr/local/bin/sudo";
+		$sudo = (isset($cmd['sudo']) && is_executable($sudo_bin)) ? "{$sudo_bin} " : "";
+		$wcmd = !empty($cmd['warning']) ? "-w {$cmd['warning']}" : "";
+		$ccmd = !empty($cmd['critical']) ? "-c {$cmd['critical']}" : "";
 		if (is_executable("{$nagios_check_path}/{$cmd['command']}"))
-			$cmds[] = "command[{$cmd['name']}]={$nagios_check_path}/{$cmd['command']} -w {$cmd['warning']} -c {$cmd['critical']} {$cmd['extra']}\n";
+			$cmds[] = "command[{$cmd['name']}]={$sudo}{$nagios_check_path}/{$cmd['command']} {$wcmd} {$ccmd} {$cmd['extra']}\n";
 	}
 	$commands = implode($cmds);
 
diff --git a/config/nrpe2/nrpe2.xml b/config/nrpe2/nrpe2.xml
index e013b47c..5b84b97f 100644
--- a/config/nrpe2/nrpe2.xml
+++ b/config/nrpe2/nrpe2.xml
@@ -3,7 +3,7 @@
 	<description>Nagios NRPEv2</description>
 	<requirements>Describe your package requirements here</requirements>
 	<name>nrpe2</name>
-	<version>2.11</version>
+	<version>2.2</version>
 	<title>NRPEv2</title>
 	<aftersaveredirect>/pkg_edit.php?xml=nrpe2.xml&amp;id=0</aftersaveredirect>
 	<include_file>/usr/local/pkg/nrpe2.inc</include_file>
-- 
cgit v1.2.3