Since 3920bis it's allowed to do the final sasl data as part of the success stanza instead of doing an extra roundtrip (challenge with final data, empty response). At the moment wocky doesn't handle this, fixed in my branch (see URL). This branch also refactors the SaslHandler stuff slightly such that it's easier to implement a trivial handler and such that handlers for mechanisms in which the server sends the final data don't have to care wether it will be received in a <success /> or a <challenge />
The branch is a whole is ok. Generalizing extracting the data from the SASL stanzas is definitely a good thing, as is separating the function for the initial data. I'm not really convinced by the "handle_auth_data" name, but I don't have a better suggestion. The WockySaslHandlerIface documentation is now out of date. Please update it.
Documentation updated as per your suggestion (reviewed by wjt). the method name is indeed not great, but i also failed to come up with something better.. Merged all the way, thanks for the review
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.