libgsasl 1.6.1 changed protocol, PSI+ can no longer connect

[Plague]

As also reported here:
http://lists.gnu.org/archive/html/help-gsasl/2011-08/msg00000.html

libgsasl changed IO protocol: "DIGEST-MD5: The server code now returns GSASL_OK after the final token."
since upgrading libgsasl, PSI+ IM client can no longer connect to the server, by reverting the change everything works.

Although this for me is a breakage in the api/protocol on gsasl side, the fastest and best way to fix the problem is to update jabberd2 to the new protocol, which AFAIK is backwards compatible.

Well read the diff on the site I linked to, it's more comprehensive than my text I think.

I am using Gentoo jabberd2-2.2.4 libgsas-1.6.1 psi 0.14-r4

[Tomasz Sterna (smoku)]

This is not a problem with jabberd2 code.
It's a problem with some clients, that do not implement SASL properly.

[Plague (4plague)]

Well, I looked at the code and since GSASL changed behavior from reading till the end, the last chunk:
from returning GSASL_NEEDS_MORE and the *next* read returns GSASL_OK
to returning GSASL_OK right away

and you have sasl_gsasl.c:378 versus sasl_gsasl.c:394 where one processes data where the other does not
AND that singe gsasl changed on the server, not client when the communication stopped working, I am still inclined to say that you should test the bug first...

[Tomasz Sterna (smoku)]

Oh. I guess for Psi receiving <success/> stanza is not enough and it's still waiting for some leftover SASL data.
Although it shouldn't be necessary according to RFC, please try the attached patch that sends the data in the finishing <success/> stanza.

[Plague (4plague)]

Thanks! After the patch it seems to work with vanilla libgsasl.
Should I post a bugreport on psi or will you? Seeing as you know more about their problem.

[Tomasz Sterna (smoku)]

I will push the patch to the jabberd2 git then.

Please report the problem to Psi, as I don't even know where to begin. ;-)

[Tomasz Sterna (smoku)]

Fixed by https://github.com/smokku/jabberd2/commit/ab31b8edb6a2df2fcef4fb4e93f94b3d7e78c241

[Tomasz Sterna (smoku)]

Fixed in 2.2.15