Refactor AbpClaimsMapMiddleware

framework/src/Volo.Abp.AspNetCore/Volo/Abp/AspNetCore/Security/Claims/AbpClaimsMapMiddleware.cs

@@ -13,15 +13,32 @@ namespace Volo.Abp.AspNetCore.Security.Claims
     {
         public async Task InvokeAsync(HttpContext context, RequestDelegate next)
         {
-            var currentPrincipalAccessor = context.RequestServices.GetRequiredService<ICurrentPrincipalAccessor>();
-            var mapOptions = context.RequestServices.GetRequiredService<IOptions<AbpClaimsMapOptions>>().Value;
+            var currentPrincipalAccessor = context.RequestServices
+                .GetRequiredService<ICurrentPrincipalAccessor>();
 
-            var mapClaims = currentPrincipalAccessor.Principal.Claims.Where(p => mapOptions.Maps.Keys.Contains(p.Type));
-            currentPrincipalAccessor.Principal.AddIdentity(new ClaimsIdentity(mapClaims.Select(p => new Claim(
-                mapOptions.Maps[p.Type],
-                p.Value,
-                p.ValueType,
-                p.Issuer))));
+            var mapOptions = context.RequestServices
+                .GetRequiredService<IOptions<AbpClaimsMapOptions>>().Value;
+
+            var mapClaims = currentPrincipalAccessor
+                .Principal
+                .Claims
+                .Where(claim => mapOptions.Maps.Keys.Contains(claim.Type));
+
+            currentPrincipalAccessor
+                .Principal
+                .AddIdentity(
+                    new ClaimsIdentity(
+                        mapClaims
+                            .Select(
+                                claim => new Claim(
+                                    mapOptions.Maps[claim.Type],
+                                    claim.Value,
+                                    claim.ValueType,
+                                    claim.Issuer
+                                )
+                            )
+                    )
+                );
 
             await next(context);
         }

framework/test/Volo.Abp.AspNetCore.Mvc.Tests/Volo/Abp/AspNetCore/Mvc/AbpAspNetCoreMvcTestModule.cs

@@ -1,10 +1,7 @@
 using System;
-using System.Collections.Generic;
-using System.Linq;
 using System.Security.Claims;
 using Localization.Resources.AbpUi;
 using Microsoft.AspNetCore.Builder;
-using Microsoft.AspNetCore.Mvc.ApplicationModels;
 using Microsoft.AspNetCore.Mvc.RazorPages;
 using Microsoft.Extensions.DependencyInjection;
 using Volo.Abp.AspNetCore.Mvc.Authorization;
@@ -16,7 +13,6 @@ using Volo.Abp.Autofac;
 using Volo.Abp.Localization;
 using Volo.Abp.MemoryDb;
 using Volo.Abp.Modularity;
-using Volo.Abp.Security.Claims;
 using Volo.Abp.TestApp;
 using Volo.Abp.Validation.Localization;
 using Volo.Abp.VirtualFileSystem;