enhanced (and corrected) the TCP testing of apache

vogelor

2011-04-01 e5c5affc01a0559139d6aea8b5f58052cfa46955

enhanced (and corrected) the TCP testing of apache

 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;
        }
    }