Skip to content

Commit 2b12b59

Browse files
committed
refactor: Use interface for AuthTokenService
- Provides abstraction - Improves testability
1 parent e45efff commit 2b12b59

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

lib/src/middlewares/authentication_middleware.dart

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import 'package:dart_frog/dart_frog.dart';
2+
// Import the interface type
23
import 'package:ht_api/src/services/auth_token_service.dart';
34
import 'package:ht_shared/ht_shared.dart';
45

@@ -18,6 +19,7 @@ Middleware authenticationProvider() {
1819
return (handler) {
1920
return (context) async {
2021
print('[AuthMiddleware] Entered.'); // Log 1: Entry
22+
// Read the interface type
2123
AuthTokenService tokenService;
2224
try {
2325
print('[AuthMiddleware] Attempting to read AuthTokenService...'); // Log 2: Before read

routes/_middleware.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -232,8 +232,8 @@ Handler middleware(Handler handler) {
232232
// --- Provide Auth Dependencies ---
233233
.use(provider<HtDataRepository<User>>((_) => userRepository))
234234
.use(provider<HtEmailRepository>((_) => emailRepository))
235-
// Provide the concrete JwtAuthTokenService type for diagnosis
236-
.use(provider<JwtAuthTokenService>((_) => authTokenService))
235+
// Provide the AuthTokenService interface type
236+
.use(provider<AuthTokenService>((_) => authTokenService))
237237
.use(
238238
provider<VerificationCodeStorageService>(
239239
(_) => verificationCodeStorageService,
@@ -244,9 +244,9 @@ Handler middleware(Handler handler) {
244244
.use(provider<Uuid>((_) => uuid)) // Provide Uuid instance
245245

246246
// --- Core Middleware ---
247-
.use(requestLogger()) // Basic request logging
248-
// Apply authenticationProvider to make User? available downstream
247+
// Apply authenticationProvider first (after providers)
249248
.use(authenticationProvider())
249+
.use(requestLogger()) // Then basic request logging
250250
// Error handler should generally be last to catch all upstream errors
251251
.use(errorHandler());
252252
}

0 commit comments

Comments
 (0)