fix: May not add CasbinAuthorizationHandler in some situations

Signed-off-by: Sagilio <[email protected]>
casbin-net · Mar 23, 2021 · e5dc077 · e5dc077
1 parent a06439c
commit e5dc077
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/Casbin.AspNetCore.Core/CoreServiceCollectionExtension.cs b/src/Casbin.AspNetCore.Core/CoreServiceCollectionExtension.cs
@@ -50,10 +50,13 @@ public static IServiceCollection AddCasbinAuthorizationCore(
             services.TryAddSingleton<ICasbinAuthorizationContextFactory, DefaultCasbinAuthorizationContextFactory>();
             services.TryAddScoped<IEnforceService, DefaultEnforcerService>();
             services.TryAddSingleton<IRequestTransformersCache, RequestTransformersCache>();
-            services.TryAddScoped<IAuthorizationHandler, CasbinAuthorizationHandler>();
-            services.TryAddSingleton<IRequestTransformer, BasicRequestTransformer>();
-            services.TryAddSingleton<IRequestTransformer, RbacRequestTransformer>();
-            services.TryAddSingleton<IRequestTransformer, KeyMatchRequestTransformer>();
+
+            // Can not change to TryAdd, because there interface may need more than one implement.
+            services.AddScoped<IAuthorizationHandler, CasbinAuthorizationHandler>();
+            services.AddSingleton<IRequestTransformer, BasicRequestTransformer>();
+            services.AddSingleton<IRequestTransformer, RbacRequestTransformer>();
+            services.AddSingleton<IRequestTransformer, KeyMatchRequestTransformer>();
+
             services.AddAuthorizationCore();
             return services;
         }