refactor(auth): improve authentication logging

- Added detailed logging
- Improved error handling

Author: fulleni

lib/src/middlewares/authentication_middleware.dart

@@ -23,43 +23,53 @@ Middleware authenticationProvider() {
       AuthTokenService tokenService;
       try {
         print(
-            '[AuthMiddleware] Attempting to read AuthTokenService...'); // Log 2: Before read
+          '[AuthMiddleware] Attempting to read AuthTokenService...',
+        ); // Log 2: Before read
         tokenService = context.read<AuthTokenService>();
         print(
-            '[AuthMiddleware] Successfully read AuthTokenService.'); // Log 3: After read
+          '[AuthMiddleware] Successfully read AuthTokenService.',
+        ); // Log 3: After read
       } catch (e, s) {
         print(
-            '[AuthMiddleware] FAILED to read AuthTokenService: $e\n$s'); // Log Error
+          '[AuthMiddleware] FAILED to read AuthTokenService: $e\n$s',
+        ); // Log Error
         // Re-throw the error to be caught by the main error handler
         rethrow;
       }
       User? user; // Initialize user as null
 
       // Extract the Authorization header
       print(
-          '[AuthMiddleware] Attempting to read Authorization header...'); // Log 4: Before header read
+        '[AuthMiddleware] Attempting to read Authorization header...',
+      ); // Log 4: Before header read
       final authHeader = context.request.headers['Authorization'];
       print(
-          '[AuthMiddleware] Authorization header value: $authHeader'); // Log 5: Header value
+        '[AuthMiddleware] Authorization header value: $authHeader',
+      ); // Log 5: Header value
 
       if (authHeader != null && authHeader.startsWith('Bearer ')) {
         // Extract the token string
         final token = authHeader.substring(7); // Length of 'Bearer '
         print(
-            '[AuthMiddleware] Extracted Bearer token.'); // Log 6: Token extracted
+          '[AuthMiddleware] Extracted Bearer token.',
+        ); // Log 6: Token extracted
         try {
           print(
-              '[AuthMiddleware] Attempting to validate token...'); // Log 7: Before validate
+            '[AuthMiddleware] Attempting to validate token...',
+          ); // Log 7: Before validate
           // Validate the token using the service
           user = await tokenService.validateToken(token);
           print(
-              '[AuthMiddleware] Token validation returned: ${user?.id ?? 'null'}'); // Log 8: After validate
+            '[AuthMiddleware] Token validation returned: ${user?.id ?? 'null'}',
+          ); // Log 8: After validate
           if (user != null) {
             print(
-                '[AuthMiddleware] Authentication successful for user: ${user.id}');
+              '[AuthMiddleware] Authentication successful for user: ${user.id}',
+            );
           } else {
             print(
-                '[AuthMiddleware] Invalid token provided (validateToken returned null).');
+              '[AuthMiddleware] Invalid token provided (validateToken returned null).',
+            );
             // Optional: Could throw UnauthorizedException here if *all* routes
             // using this middleware strictly require a valid token.
             // However, providing null allows routes to handle optional auth.
@@ -73,7 +83,8 @@ Middleware authenticationProvider() {
         } catch (e, s) {
           // Catch unexpected errors during validation
           print(
-              '[AuthMiddleware] Unexpected error during token validation: $e\n$s');
+            '[AuthMiddleware] Unexpected error during token validation: $e\n$s',
+          );
           user = null; // Keep user null if unexpected error occurred
         }
       } else {
@@ -83,7 +94,8 @@ Middleware authenticationProvider() {
       // Provide the User object (or null) into the context
       // This makes `context.read<User?>()` available downstream.
       print(
-          '[AuthMiddleware] Providing User (${user?.id ?? 'null'}) to context.'); // Log 9: Before provide
+        '[AuthMiddleware] Providing User (${user?.id ?? 'null'}) to context.',
+      ); // Log 9: Before provide
       return handler(context.provide<User?>(() => user));
     };
   };

lib/src/services/simple_auth_token_service.dart

@@ -46,13 +46,15 @@ class SimpleAuthTokenService implements AuthTokenService {
 
     try {
       print(
-          '[SimpleAuthTokenService] Attempting to read user from repository...');
+        '[SimpleAuthTokenService] Attempting to read user from repository...',
+      );
       final user = await _userRepository.read(userId);
       print('[SimpleAuthTokenService] User read successful: ${user.id}');
       return user;
     } on NotFoundException {
       print(
-          '[SimpleAuthTokenService] Validation failed: User ID $userId not found.');
+        '[SimpleAuthTokenService] Validation failed: User ID $userId not found.',
+      );
       // Return null if user not found, mimicking successful validation
       // of a token for a non-existent user. The middleware handles this.
       return null;

routes/_middleware.dart

@@ -250,11 +250,17 @@ Handler middleware(Handler handler) {
       .use(provider<HtDataRepository<Category>>((_) => categoryRepository))
       .use(provider<HtDataRepository<Source>>((_) => sourceRepository))
       .use(provider<HtDataRepository<Country>>((_) => countryRepository))
-      .use(provider<HtDataRepository<User>>(
-          (_) => userRepository)) // Used by Auth services
+      .use(
+        provider<HtDataRepository<User>>(
+          (_) => userRepository,
+        ),
+      ) // Used by Auth services
       .use(provider<HtAppSettingsRepository>((_) => settingsRepository))
-      .use(provider<HtEmailRepository>(
-          (_) => emailRepository)) // Used by AuthService
+      .use(
+        provider<HtEmailRepository>(
+          (_) => emailRepository,
+        ),
+      ) // Used by AuthService
 
       // --- 4. Authentication Service Providers (Auth Logic Dependencies) ---
       // PURPOSE: Provide the core services needed for authentication logic.
@@ -264,15 +270,21 @@ Handler middleware(Handler handler) {
       //          - `AuthService` uses several repositories and `AuthTokenService`.
       //          - `VerificationCodeStorageService` is used by `AuthService`.
       //          - `Uuid` is used by `AuthService` and `JwtAuthTokenService`.
-      .use(provider<AuthTokenService>(
-          (_) => authTokenService)) // Read by authenticationProvider
+      .use(
+        provider<AuthTokenService>(
+          (_) => authTokenService,
+        ),
+      ) // Read by authenticationProvider
       .use(
         provider<VerificationCodeStorageService>(
           (_) => verificationCodeStorageService,
         ),
       ) // Read by AuthService
-      .use(provider<AuthService>(
-          (_) => authService)) // Reads other services/repos
+      .use(
+        provider<AuthService>(
+          (_) => authService,
+        ),
+      ) // Reads other services/repos
       .use(provider<Uuid>((_) => uuid)) // Read by AuthService & TokenService
 
       // --- 5. Authentication Middleware (User Context Population) ---