The Channel.Interface.SASLAuthentication has an AbortSASL() method that allow to stop the SASL process. The first argument is the reason to abort and currently it can be Invalid_Challenge or User_Abort. Suppose the auth channel handler does the actual authentication through an external authentication service and this fails for some other reason (for instance a network error), what is it going to pass to AbortSASL()? Also notice that the reason is not used for anything else; once the CM gets the reason it doesn't do much with it. We should get a way to properly pass a real error to the CM when we call AbortSASL() and be able to propagate it to the other components. Considering that the reason argument is basically useless I propose to deprecate AbortSASL() and add a new TerminateSASL(s: Error, s: Debug_Message), where Error is a normal D-Bus error string. The SASLError property would be then set to Error. For compatibility CMs can implement AbortSASL(Invalid_Challenge, msg) as equivalent to TerminateSASL("ofT.Error.ServiceConfused", msg) and AbortSASL(User_Abort, msg) as "ofT.Error.Cancelled", msg). Comments? (As a side note, ar the moment the SASLError property seems to be useless in Gabble as it always says it was an authentication error.)
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/telepathy/telepathy-spec/issues/122.
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.