aboutsummaryrefslogtreecommitdiff
path: root/src/client/QXmppOutgoingClient.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'src/client/QXmppOutgoingClient.cpp')
-rw-r--r--src/client/QXmppOutgoingClient.cpp15
1 files changed, 8 insertions, 7 deletions
diff --git a/src/client/QXmppOutgoingClient.cpp b/src/client/QXmppOutgoingClient.cpp
index 1fcb7c28..aef68070 100644
--- a/src/client/QXmppOutgoingClient.cpp
+++ b/src/client/QXmppOutgoingClient.cpp
@@ -451,8 +451,12 @@ void QXmppOutgoingClient::handleStanza(const QDomElement &nodeRecv)
if (saslAvailable && configuration().useSASLAuthentication())
{
// supported and preferred SASL auth mechanisms
- QStringList supportedMechanisms = QXmppSaslClient::availableMechanisms();
const QString preferredMechanism = configuration().saslAuthMechanism();
+ QStringList supportedMechanisms = QXmppSaslClient::availableMechanisms();
+ if (supportedMechanisms.contains(preferredMechanism)) {
+ supportedMechanisms.removeAll(preferredMechanism);
+ supportedMechanisms.prepend(preferredMechanism);
+ }
if (configuration().facebookAppId().isEmpty() || configuration().facebookAccessToken().isEmpty())
supportedMechanisms.removeAll("X-FACEBOOK-PLATFORM");
if (configuration().windowsLiveAccessToken().isEmpty())
@@ -463,19 +467,16 @@ void QXmppOutgoingClient::handleStanza(const QDomElement &nodeRecv)
// determine SASL Authentication mechanism to use
QStringList commonMechanisms;
QString usedMechanism;
- foreach (const QString &mechanism, features.authMechanisms()) {
- if (supportedMechanisms.contains(mechanism))
+ foreach (const QString &mechanism, supportedMechanisms) {
+ if (features.authMechanisms().contains(mechanism))
commonMechanisms << mechanism;
}
if (commonMechanisms.isEmpty()) {
warning("No supported SASL Authentication mechanism available");
disconnectFromHost();
return;
- } else if (!commonMechanisms.contains(preferredMechanism)) {
- info(QString("Desired SASL Auth mechanism '%1' is not available, selecting first available one").arg(preferredMechanism));
- usedMechanism = commonMechanisms.first();
} else {
- usedMechanism = preferredMechanism;
+ usedMechanism = commonMechanisms.first();
}
d->saslClient = QXmppSaslClient::create(usedMechanism, this);