From dfcd553fdcf9556596506ffda66d716f6db5dc6b Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Fri, 01 Apr 2011 04:13:42 -0400
Subject: [PATCH] Fixed bug in BIND monitor.

---
 server/lib/classes/monitor_tools.inc.php |  120 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 1 files changed, 112 insertions(+), 8 deletions(-)

diff --git a/server/lib/classes/monitor_tools.inc.php b/server/lib/classes/monitor_tools.inc.php
index 1e33ea7..a9bdaed 100644
--- a/server/lib/classes/monitor_tools.inc.php
+++ b/server/lib/classes/monitor_tools.inc.php
@@ -332,7 +332,7 @@
 
 		/* the ISPC-Version has no state. It is, what it is */
 		$state = 'no_state';
-		
+
 		/*
 		 * Return the Result
 		 */
@@ -446,7 +446,7 @@
 		 * maybe someone knows better...???...
 		 */
 		$state = 'no_state';
-		
+
 		/*
 		 * Return the Result
 		 */
@@ -518,8 +518,16 @@
 		/** the id of the server as int */
 		$server_id = intval($conf['server_id']);
 
-		/** get the "active" Services of the server from the DB */
-		$services = $app->dbmaster->queryOneRecord('SELECT * FROM server WHERE server_id = ' . $server_id);
+		/**  get the "active" Services of the server from the DB */
+		$services = $app->db->queryOneRecord('SELECT * FROM server WHERE server_id = ' . $server_id);
+		/*
+		 * If the DB is down, we have to set the db to "yes".
+		 * If we don't do this, then the monitor will NOT monitor, that the db is down and so the
+		 * rescue-module can not try to rescue the db
+		 */
+		if ($services == null) {
+			$services['db_server'] = 1;
+		}
 
 		/* The type of the Monitor-data */
 		$type = 'services';
@@ -589,7 +597,7 @@
 		/* Monitor BIND-Server */
 		$data['bindserver'] = -1; // unknown - not needed
 		if ($services['dns_server'] == 1) {
-			if ($this->_checkTcp('localhost', 53)) {
+			if ($this->_checkUdp('localhost', 53)) {
 				$data['bindserver'] = 1;
 			} else {
 				$data['bindserver'] = 0;
@@ -607,7 +615,7 @@
 				$state = 'error'; // because service is down
 			}
 		}
-		
+
 		/*
 		 * Return the Result
 		 */
@@ -791,6 +799,28 @@
 				$state = 'info';
 				$data['output'] = shell_exec('glsa-check -pv --nocolor affected 2>/dev/null');
 			}
+		} elseif (file_exists('/etc/SuSE-release')) {
+
+			/*
+			 * update and find the upgrade.
+			 * if there is any output, then there is a needed update
+			 */
+			$aptData = shell_exec('zypper -q lu');
+			if ($aptData == '') {
+				/* There is nothing to update! */
+				$state = 'ok';
+			} else {
+				/*
+				 * There is something to update! this is in most cases not critical, so we can
+				 * do a system-update once a month or so...
+				 */
+				$state = 'info';
+			}
+
+			/*
+			 * Fetch the output
+			 */
+			$data['output'] = shell_exec('zypper --non-interactive up');
 		} else {
 			/*
 			 * It is not Debian/Ubuntu, so there is no data and no state
@@ -1070,6 +1100,7 @@
 	}
 
 	public function monitorSysLog() {
+		global $app;
 		global $conf;
 
 		/* the id of the server as int */
@@ -1512,10 +1543,22 @@
 		$fp = @fsockopen($host, $port, $errno, $errstr, 2);
 
 		if ($fp) {
+			/*
+			 * We got a connection, but maybe apache is not able to send data over this
+			 * connection?
+			 */
+			fwrite($fp, "GET / HTTP/1.0\r\n\r\n");
+			stream_set_timeout($fp, 2);
+			$res = fread($fp, 10);
+			$info = stream_get_meta_data($fp);
 			fclose($fp);
-			return true;
+			if ($info['timed_out']) {
+				return false; // Apache was not able to send data over this connection
+			} else {
+				return true; // Apache was able to send data over this connection
+			}
 		} else {
-			return false;
+			return false; // Apache was not able to establish a connection
 		}
 	}
 
@@ -1542,6 +1585,67 @@
 			return false;
 		}
 	}
+	
+    /*
+     * Set the state to the given level (or higher, but not lesser).
+     * * If the actual state is critical and you call the method with ok,
+     *   then the state is critical.
+     *
+     * * If the actual state is critical and you call the method with error,
+     *   then the state is error.
+     */
+    private function _setState($oldState, $newState)
+    {
+        /*
+         * Calculate the weight of the old state
+         */
+        switch ($oldState) {
+            case 'no_state': $oldInt = 0;
+                break;
+            case 'ok': $oldInt = 1;
+                break;
+            case 'unknown': $oldInt = 2;
+                break;
+            case 'info': $oldInt = 3;
+                break;
+            case 'warning': $oldInt = 4;
+                break;
+            case 'critical': $oldInt = 5;
+                break;
+            case 'error': $oldInt = 6;
+                break;
+        }
+        /*
+         * Calculate the weight of the new state
+         */
+        switch ($newState) {
+            case 'no_state': $newInt = 0 ;
+                break;
+            case 'ok': $newInt = 1 ;
+                break;
+            case 'unknown': $newInt = 2 ;
+                break;
+            case 'info': $newInt = 3 ;
+                break;
+            case 'warning': $newInt = 4 ;
+                break;
+            case 'critical': $newInt = 5 ;
+                break;
+            case 'error': $newInt = 6 ;
+                break;
+        }
+
+        /*
+         * Set to the higher level
+         */
+        if ($newInt > $oldInt){
+            return $newState;
+        }
+        else
+        {
+            return $oldState;
+        }
+    }
 
 	private function _getIntArray($line) {
 		/** The array of float found */

--
Gitblit v1.9.1