fix: sentry auth exception (#1387)

NIAEFEUP · Nov 11, 2024 · 1da2fbf · 1da2fbf
2 parents d8a057e + bc5ba1c
commit 1da2fbf
Show file tree

Hide file tree

Showing 5 changed files with 56 additions and 40 deletions.
diff --git a/packages/uni_app/lib/model/entities/login_exceptions.dart b/packages/uni_app/lib/model/entities/login_exceptions.dart
diff --git a/packages/uni_app/lib/session/exception.dart b/packages/uni_app/lib/session/exception.dart
@@ -1,4 +1,5 @@
 enum AuthenticationExceptionType {
+  internetError,
   wrongCredentials,
   expiredCredentials,
   other,

diff --git a/packages/uni_app/lib/session/flows/credentials/request.dart b/packages/uni_app/lib/session/flows/credentials/request.dart
@@ -1,3 +1,4 @@
+import 'dart:async';
 import 'package:http/http.dart' as http;
 import 'package:uni/controller/fetchers/faculties_fetcher.dart';
 import 'package:uni/session/exception.dart';
@@ -35,11 +36,20 @@ class CredentialsSessionRequest extends SessionRequest {
           'Failed to authenticate user',
           AuthenticationExceptionType.expiredCredentials,
         );
-      } else {
+      } else if (failureReason == LoginFailureReason.internetError) {
+        throw const AuthenticationException(
+          'Failed to authenticate user',
+          AuthenticationExceptionType.internetError,
+        );
+      } else if (failureReason == LoginFailureReason.wrongCredentials) {
         throw const AuthenticationException(
           'Failed to authenticate user',
           AuthenticationExceptionType.wrongCredentials,
         );
+      } else {
+        throw const AuthenticationException(
+          'Failed to authenticate user',
+        );
       }
     }
 

diff --git a/packages/uni_app/lib/sigarra/endpoints/html/authentication/login/response.dart b/packages/uni_app/lib/sigarra/endpoints/html/authentication/login/response.dart
@@ -8,6 +8,7 @@ class LoginResponse extends SigarraResponse {
 
 enum LoginFailureReason {
   serverError,
+  internetError,
   wrongCredentials,
   expiredCredentials,
   blockedAccount,

diff --git a/packages/uni_app/lib/view/login/login.dart b/packages/uni_app/lib/view/login/login.dart
@@ -9,9 +9,9 @@ import 'package:sentry_flutter/sentry_flutter.dart';
 import 'package:uni/app_links/uni_app_links.dart';
 import 'package:uni/controller/networking/url_launcher.dart';
 import 'package:uni/generated/l10n.dart';
-import 'package:uni/model/entities/login_exceptions.dart';
 import 'package:uni/model/providers/startup/session_provider.dart';
 import 'package:uni/model/providers/state_providers.dart';
+import 'package:uni/session/exception.dart';
 import 'package:uni/session/flows/credentials/initiator.dart';
 import 'package:uni/session/flows/federated/initiator.dart';
 import 'package:uni/utils/constants.dart';
@@ -99,36 +99,51 @@ class LoginPageViewState extends State<LoginPageView>
             _loggingIn = false;
           });
         }
-      } catch (err, st) {
+      } on AuthenticationException catch (err, st) {
         setState(() {
           _loggingIn = false;
         });
-        if (err is ExpiredCredentialsException) {
-          _updatePasswordDialog();
-        } else if (err is InternetStatusException) {
-          if (mounted) {
-            unawaited(
-              ToastMessage.warning(
-                context,
-                S.of(context).internet_status_exception,
-              ),
-            );
-          }
-        } else if (err is WrongCredentialsException) {
-          if (mounted) {
-            unawaited(
-              ToastMessage.error(
-                context,
-                S.of(context).wrong_credentials_exception,
-              ),
-            );
-          }
-        } else {
-          Logger().e(err, stackTrace: st);
-          unawaited(Sentry.captureException(err, stackTrace: st));
-          if (mounted) {
-            unawaited(ToastMessage.error(context, S.of(context).failed_login));
-          }
+
+        switch (err.type) {
+          case AuthenticationExceptionType.expiredCredentials:
+            _updatePasswordDialog();
+          case AuthenticationExceptionType.internetError:
+            if (mounted) {
+              unawaited(
+                ToastMessage.warning(
+                  context,
+                  S.of(context).internet_status_exception,
+                ),
+              );
+            }
+          case AuthenticationExceptionType.wrongCredentials:
+            if (mounted) {
+              unawaited(
+                ToastMessage.error(
+                  context,
+                  S.of(context).wrong_credentials_exception,
+                ),
+              );
+            }
+          default:
+            Logger().e(err, stackTrace: st);
+            unawaited(Sentry.captureException(err, stackTrace: st));
+            if (mounted) {
+              unawaited(
+                ToastMessage.error(context, S.of(context).failed_login),
+              );
+            }
+        }
+      }
+      // Handles other unexpected exceptions
+      catch (err, st) {
+        setState(() {
+          _loggingIn = false;
+        });
+        Logger().e(err, stackTrace: st);
+        unawaited(Sentry.captureException(err, stackTrace: st));
+        if (mounted) {
+          unawaited(ToastMessage.error(context, S.of(context).failed_login));
         }
       }
     }