enhanced (and corrected) the TCP testing of apache

vogelor

2011-04-01 e5c5affc01a0559139d6aea8b5f58052cfa46955

 server/lib/classes/monitor_tools.inc.php

@@ -1539,26 +1539,36 @@
   }

   private function _checkTcp($host, $port) {

      /* Try to open a connection */
      $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?
          * We got a connection, this means, everything is O.K.
          * But maybe we are able to do more deep testing?
          */
         if ($port == 80) {
            /*
             * Port 80 means, testing APACHE
             * So we can do a deepter test and try to get data over this connection.
             * (if apache hangs, we get a connection but a timeout by trying to GET the data!)
          */
         fwrite($fp, "GET / HTTP/1.0\r\n\r\n");
         stream_set_timeout($fp, 2);
         $res = fread($fp, 10);
            stream_set_timeout($fp, 5); // Timeout after 5 seconds
            $res = fread($fp, 10);  // try to get 10 bytes (enough to test!)
         $info = stream_get_meta_data($fp);
         fclose($fp);
         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
         }
         }

         /* The connection is no longer needed */
         fclose($fp);
         /* We are able to establish a connection */
         return true;
      } else {
         return false; // Apache was not able to establish a connection
         /* We are NOT able to establish a connection */
         return false;
      }
   }

@@ -1586,7 +1596,7 @@
      }
   }
   
    /*
   /**
     * 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.
@@ -1594,8 +1604,7 @@
     * * If the actual state is critical and you call the method with error,
     *   then the state is error.
     */
    private function _setState($oldState, $newState)
    {
   private function _setState($oldState, $newState) {
        /*
         * Calculate the weight of the old state
         */
@@ -1640,9 +1649,7 @@
         */
        if ($newInt > $oldInt){
            return $newState;
        }
        else
        {
      } else {
            return $oldState;
        }
    }