-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ping returns error #97
Comments
That looks like what is going on to me as well. I'm not really sure what to do about that, as the ping requests are used to keep the connection alive. Ejabberd and OpenFire close connections without those ping requests. |
Is there a way to turn them off? I'm running the Cisco Jabber server and from what I can see there's no need for the ping requests to keep the connection open. It would sure help stop the logs from filling up! Sent from my iPhone
|
There isn't presently a way to turn off the ping requests. |
We also get these error message, running an ancient ejabberd. |
This seems to stop the ping error messages as per xmppjs#97
Stephen given I’m very new to this, where did you add the id: ‘hubby-xmpp’ line? I assume: ping = new ltx.Element('iq', type: 'get') I still get the following: ERROR [xmpp error]
|
I had already deleted my comment about id, it doesn't fix these ping errors. |
@beerygaz @shumphrey Have you guys seen my pull request (#93) that addresses this issue? |
@pixelrebel I believe the two are separate issues, I added an id to the ping request and still get the errors. As far as I can tell, your PR fixes an error where the jabber server understands ping but needs an id, and this error is where the jabber server doesn't understand pings at all? This error message says "service unavailable", yours says "missing 'id'" |
I'm very new to trying to work with both hubot and XMPP, but this looks to me like my XMPP server does not support the ping service?
The text was updated successfully, but these errors were encountered: