Open DavidusEdwardus opened 11 years ago
I don't mind adding in jabber:iq:auth as long as it's advertised as a a feature in stream:features and can be detected.
Note that it's not part of XMPP 1.0 at all.
Oh I've misunderstood your patch. This causes it to keep looping waiting for stream:features?
Indeed, its looking for the mechanism tag, and its ostensible (and in my testing with the SASL module disabled :P ) , that the
Of course as you say, its not part of the spec, and its also deprecated, but for completeness, that was it.
When writing my own XMPP service using DJabbberd , I found i could log in using BOSH and Pidgin, but not strophejs. jabber:iq:auth is deprecated, but when testing, I had SASL disabled.
At what point a server advertises the iq-auth stream feature is up to the implementation or deployment, and in my case, this is as per Example 9 Xep-0078. The code looks for the mechanism tag, and if not found, loops infinitely in the case this is never offered.
My fix may not be relevant or suitable, but here it is for completeness or if anyone else is curious
From 2958 on