'[Authorize(Roles = "admin" )]' MVC 和 Azure Active Directory B2C ASP.NET 无限循环

Question

我试图只允许具有"全局管理员"角色的Azure Active Directory B2C用户访问以下类(这就是我包含Authorize命令的原因)：

[Authorize(Roles = "admin")]
public class UserProfileController : Controller
{
... controller class ...
}

我的启动类看起来像这样：

public partial class Startup
{
private static string clientId = ConfigurationManager.AppSettings["ida:ClientId"];
private static string appKey = ConfigurationManager.AppSettings["ida:ClientSecret"];
private static string aadInstance = ConfigurationManager.AppSettings["ida:AADInstance"];
private static string tenantId = ConfigurationManager.AppSettings["ida:TenantId"];
private static string postLogoutRedirectUri = ConfigurationManager.AppSettings["ida:PostLogoutRedirectUri"];
// This is the resource ID of the AAD Graph API.  We'll need this to request a token to call the Graph API.
private static string graphResourceId = "https://graph.microsoft.com";
private static readonly string Authority = aadInstance + tenantId;
public static GraphServiceClient graphClient = null;
public static GraphServiceClient GetGraphServiceClient()
{
return graphClient;
}
public void ConfigureAuth(IAppBuilder app)
{
app.SetDefaultSignInAsAuthenticationType(CookieAuthenticationDefaults.AuthenticationType);
app.UseCookieAuthentication(new CookieAuthenticationOptions()
{
CookieSecure = CookieSecureOption.Always
});
app.UseOpenIdConnectAuthentication(
new OpenIdConnectAuthenticationOptions
{
ClientId = clientId,
Authority = Authority,
PostLogoutRedirectUri = postLogoutRedirectUri,
Notifications = new OpenIdConnectAuthenticationNotifications()
{
// If there is a code in the OpenID Connect response, redeem it for an access token and refresh token, and store those away.
AuthorizationCodeReceived = (context) => 
{
var code = context.Code;
ClientCredential credential = new ClientCredential(clientId, appKey);
string signedInUserID = context.AuthenticationTicket.Identity.FindFirst(ClaimTypes.NameIdentifier).Value;
TokenCache userTokenCache = new ADALTokenCache(signedInUserID);
AuthenticationContext authContext = new AuthenticationContext(Authority, userTokenCache);
AuthenticationResult result = authContext.AcquireTokenByAuthorizationCode(
code, new Uri(HttpContext.Current.Request.Url.GetLeftPart(UriPartial.Path)), credential, graphResourceId);
string token = result.AccessToken;
try
{
graphClient = new GraphServiceClient(
new DelegateAuthenticationProvider(
(requestMessage) =>
{
requestMessage.Headers.Authorization = new System.Net.Http.Headers.AuthenticationHeaderValue("bearer", token);
return Task.FromResult(0);
}));
}
catch (Exception e)
{
System.Diagnostics.Debug.WriteLine("Failed to create graph client: " + e.Message);
}
return Task.FromResult(0);
}
}
});
}
}

问题是：当我单击实例化UserProfileController的按钮时，AuthorizationCodeReceived = (context) =>行代码中的代码会在无限循环中一次又一次地调用。如何修复无限循环，以便只有 Azure Active Directory B2C"全局管理员"才能实例化用户配置文件控制器？

Answer 1

[授权(角色 = "管理员")]

由于使用 Authorization 属性检查用户的角色，因此需要确保当前用户的声明具有有效的角色声明。可以利用以下代码片段来检查当前用户声明：

return Json((User.Identity as ClaimsIdentity).Claims.Select(c => new { key = c.Type, value = c.Value }),JsonRequestBehavior.AllowGet);

问题是：当我单击实例化UserProfileController的按钮时，AuthorizationCodeReceived = (context) =>行代码中的代码在无限循环中一次又一次地被调用。

您可以重写 AuthorizeAttribute 下的HandleUnauthorizedRequest方法，并按如下方式定义自定义授权属性：

public class MyAuthorize : AuthorizeAttribute
{
protected override void HandleUnauthorizedRequest(System.Web.Mvc.AuthorizationContext filterContext)
{
filterContext.Result = new ContentResult() { Content = "You don't have rights to take actions" };
}
}

然后，您可以按如下方式装饰UserProfileController控制器：

[MyAuthorize(Roles = "admin")]
public class UserProfileController : Controller
{
//TODO:
}

我尝试仅允许具有"全局管理员"角色的 Azure 活动目录 B2C 用户访问以下类

在AuthorizationCodeReceived委托方法下，检索访问令牌后，需要利用 Microsoft Graph 客户端库来检查当前用户是否为全局管理员/公司管理员。如果当前用户是全局管理员/公司管理员，则需要指定角色声明，如下所示：

context.AuthenticationTicket.Identity.AddClaim(new Claim(context.AuthenticationTicket.Identity.RoleClaimType, "admin"));

注意：要检查用户是否为全局管理员，可以检索当前用户目录下的角色，然后使用 getMemberObjects API 检索组、当前用户所属的角色，然后检查全局管理员角色 ID 是否位于当前用户的成员对象中。

//List directory roles, https://developer.microsoft.com/en-us/graph/docs/api-reference/v1.0/api/directoryrole_list
var roles=await graphClient.DirectoryRoles.Request().GetAsync();
//user: getMemberObjects ,https://developer.microsoft.com/en-us/graph/docs/api-reference/v1.0/api/user_getmemberobjects

更新：

我检查了我这边的实现。下面是用于检查当前登录用户的角色的代码。

var directoryRoles = await graphClient.DirectoryRoles.Request().GetAsync();
var userRoles = await graphClient.Me.MemberOf.Request().GetAsync();
var adminRole=directoryRoles.Where(role => role.DisplayName== "Company Administrator" || role.DisplayName == "Global Administrator").FirstOrDefault();
if (userRoles.Count(role => role.Id == adminRole.Id) > 0)
{
context.AuthenticationTicket.Identity.AddClaim(new Claim(context.AuthenticationTicket.Identity.RoleClaimType, "admin"));
}
else
{
context.AuthenticationTicket.Identity.AddClaim(new Claim(context.AuthenticationTicket.Identity.RoleClaimType, "user"));
}

注意：要添加多个用户角色，可以添加多个new Claim(context.AuthenticationTicket.Identity.RoleClaimType, "<role-name>")角色声明。

这是我修改后的自定义AuthorizeAttribute：

public class MyAuthorize : AuthorizeAttribute
{
private bool noPermission = false;
public string Permissions { get; set; }
protected override bool AuthorizeCore(HttpContextBase httpContext)
{
if (!base.AuthorizeCore(httpContext))
return false;
var permissionArrs = Permissions.Trim().Split('|');
if (permissionArrs.ToList().Exists(p=>httpContext.User.IsInRole(p)))
{
return true;
}
else
{
noPermission = true;
return false;
}
}
protected override void HandleUnauthorizedRequest(System.Web.Mvc.AuthorizationContext filterContext)
{
if (noPermission)
filterContext.Result = new ContentResult() { Content = "You don't have rights to take actions" };
else
base.HandleUnauthorizedRequest(filterContext);
}
}

按如下方式装饰UserProfileController：

[MyAuthorize(Permissions = "admin|co-admin")]
public class UsersController : Controller
{
//TODO:
}