Skip to content

Commit 76589a3

Browse files
committed
fix(authorization): replace print statements with logger
- Replaced `print` statements with `Logger` calls. - Improved error handling and logging. - Enhanced debugging capabilities. - Used `Logger.severe` for configuration errors. - Used `Logger.warning` for unsupported actions.
1 parent 48cbcfe commit 76589a3

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

lib/src/middlewares/authorization_middleware.dart

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import 'package:dart_frog/dart_frog.dart';
22
import 'package:ht_api/src/rbac/permission_service.dart';
33
import 'package:ht_api/src/registry/model_registry.dart';
44
import 'package:ht_shared/ht_shared.dart';
5+
import 'package:logging/logging.dart';
6+
7+
final _log = Logger('AuthorizationMiddleware');
58

69
/// {@template authorization_middleware}
710
/// Middleware to enforce role-based permissions and model-specific access rules.
@@ -81,9 +84,9 @@ Middleware authorizationMiddleware() {
8184
final permission = requiredPermissionConfig.permission;
8285
if (permission == null) {
8386
// This indicates a configuration error in ModelRegistry
84-
print(
85-
'[AuthorizationMiddleware] Configuration Error: specificPermission '
86-
'type requires a permission string for model "$modelName", method "$method".',
87+
_log.severe(
88+
'Configuration Error: specificPermission type requires a '
89+
'permission string for model "$modelName", method "$method".',
8790
);
8891
throw const OperationFailedException(
8992
'Internal Server Error: Authorization configuration error.',
@@ -97,9 +100,9 @@ Middleware authorizationMiddleware() {
97100
case RequiredPermissionType.unsupported:
98101
// This action is explicitly marked as not supported via this generic route.
99102
// Return Method Not Allowed.
100-
print(
101-
'[AuthorizationMiddleware] Action for model "$modelName", method "$method" '
102-
'is marked as unsupported via generic route.',
103+
_log.warning(
104+
'Action for model "$modelName", method "$method" is marked as '
105+
'unsupported via generic route.',
103106
);
104107
// Throw ForbiddenException to be caught by the errorHandler
105108
throw ForbiddenException(

0 commit comments

Comments
 (0)