Simple whitespace ping implementation

Hi, while playing with XMPPHP I've found that it doesn't implement any keepalive feature and I the persistent bot being developed here was being dropped from time to time by the XMPP server (openfire, defaulting to xmpp.client.idle = 6 mins). It seems that there are two ways to fix this: 1) Send one whitespace ping at regular intervals as commented in http://xmpp.org/extensions/xep-0198.html 2) The preferred way, implement XMPP-Ping - http://xmpp.org/extensions/xep-0199.html So, being lazy as I am, I've added a quick modification, implementing whitespace pings at regular intervals (70 secs). It seems that it makes long-running clients (like bots) to work really better. Adding here the implemented modification in case it helps. Thanks for your cool library, ciao, Eloy (stronk7)

The server I'm trying to connect to doesn't pay any attention to the whitespace 
ping, so I had to do a (half-assed) implementation of XMPP-Ping. This is not a 
full implementation, only enough to send a pong back to an incoming ping.

First I implemented swa...@gmail.com's xpath_parse.patch from 
https://code.google.com/p/xmpphp/issues/detail?id=44

Then in XMPP.php, around line 136, I added:
        $this->addXPathHandler('iq/{urn:xmpp:ping}ping', 'ping_handler');

Then, same file, among the handler functions (say line 375):
    /**
    * Ping handler
    * Gets all packets matching XPath "iq/{urn:xmpp:ping}ping'
    *
    * @param string $xml
    */
    protected function ping_handler($xml) {
        $payload['from'] = $xml->attrs['from']; //lh3
        $payload['to'] = $xml->attrs['to'];     //ltlbot
        $payload['id'] = $xml->attrs['id'];     //ltlbot
        $payload['type'] = $xml->attrs['type']; //get
        if ($xml->attrs['type'] == 'get') {
            $this->send("<iq from=\"{$xml->attrs['to']}\" to=\"{$xml->attrs['from']}\" id=\"{$xml->attrs['id']}\" type=\"result\" />");
        }
    }

Deborah

Original comment by deborah....@gmail.com on 17 Jun 2014 at 8:05

skmaingi / xmpphp

Simple whitespace ping implementation #52