Fix wrong user/password state misrepresented as expired session

opengisch · Oct 19, 2023 · cd91c1e · cd91c1e
1 parent 4e2f4f4
commit cd91c1e
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/core/qfieldcloudconnection.cpp b/src/core/qfieldcloudconnection.cpp
@@ -138,7 +138,8 @@ CloudUserInformation QFieldCloudConnection::userInformation() const
 
 void QFieldCloudConnection::login()
 {
-  NetworkReply *reply = ( !mToken.isEmpty() && ( mPassword.isEmpty() || mUsername.isEmpty() ) )
+  const bool loginUsingToken = !mToken.isEmpty() && ( mPassword.isEmpty() || mUsername.isEmpty() );
+  NetworkReply *reply = loginUsingToken
                           ? get( QStringLiteral( "/api/v1/auth/user/" ) )
                           : post( QStringLiteral( "/api/v1/auth/token/" ), QVariantMap(
                                                                              {
@@ -181,14 +182,16 @@ void QFieldCloudConnection::login()
       {
         emit loginFailed( tr( "Timeout error, please retry" ) );
       }
-      else if ( httpCode == 400 )
+      else if ( httpCode == 400 || httpCode == 401 )
       {
-        emit loginFailed( tr( "Wrong username or password" ) );
-      }
-      else if ( httpCode == 401 )
-      {
-        emit loginFailed( tr( "Session expired" ) );
-        invalidateToken();
+        if ( !loginUsingToken )
+        {
+          emit loginFailed( tr( "Wrong username or password" ) );
+        }
+        else
+        {
+          emit loginFailed( tr( "Session expired" ) );
+        }
       }
       else
       {