Cleanup

Switch to logging interfaces reference
Tweak DenyAnonymous logic

Fixes #181
Fixes #169

Author: HaoK

src/Microsoft.AspNet.Authentication/project.json

@@ -6,7 +6,7 @@
         "Microsoft.AspNet.RequestContainer": "1.0.0-*",
         "Microsoft.AspNet.Http.Interfaces": "1.0.0-*",
         "Microsoft.AspNet.Http.Core": "1.0.0-*",
-        "Microsoft.Framework.Logging": "1.0.0-*",
+        "Microsoft.Framework.Logging.Interfaces": "1.0.0-*",
         "Microsoft.Framework.NotNullAttribute.Internal": { "type": "build", "version": "1.0.0-*" }
     },
     "frameworks": {

src/Microsoft.AspNet.Authorization/DefaultAuthorizationService.cs

@@ -23,11 +23,9 @@ public DefaultAuthorizationService(IOptions<AuthorizationOptions> options, IEnum
         public bool Authorize(ClaimsPrincipal user, object resource, string policyName)
         {
             var policy = _options.GetPolicy(policyName);
-            if (policy == null)
-            {
-                return false;
-            }
-            return this.Authorize(user, resource, policy);
+            return (policy == null) 
+                ? false 
+                : this.Authorize(user, resource, policy);
         }
 
         public bool Authorize(ClaimsPrincipal user, object resource, params IAuthorizationRequirement[] requirements)
@@ -53,11 +51,9 @@ public async Task<bool> AuthorizeAsync(ClaimsPrincipal user, object resource, pa
         public Task<bool> AuthorizeAsync(ClaimsPrincipal user, object resource, string policyName)
         {
             var policy = _options.GetPolicy(policyName);
-            if (policy == null)
-            {
-                return Task.FromResult(false);
-            }
-            return this.AuthorizeAsync(user, resource, policy);
+            return (policy == null)
+                ? Task.FromResult(false)
+                : this.AuthorizeAsync(user, resource, policy);
         }
     }
 }

src/Microsoft.AspNet.Authorization/DenyAnonymousAuthorizationHandler.cs

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Open Technologies, Inc. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System.Threading.Tasks;
+using System.Linq;
 
 namespace Microsoft.AspNet.Authorization
 {
@@ -11,9 +11,8 @@ public override void Handle(AuthorizationContext context, DenyAnonymousAuthoriza
         {
             var user = context.User;
             var userIsAnonymous =
-                user == null ||
-                user.Identity == null ||
-                !user.Identity.IsAuthenticated;
+                user?.Identity == null ||
+                !user.Identities.Any(i => i.IsAuthenticated);
             if (!userIsAnonymous)
             {
                 context.Succeed(requirement);

src/Microsoft.AspNet.Authorization/project.json

@@ -3,7 +3,7 @@
     "description": "ASP.NET 5 authorization classes.",
     "dependencies": {
         "Microsoft.AspNet.Http.Interfaces": "1.0.0-*",
-        "Microsoft.Framework.Logging": "1.0.0-*",
+        "Microsoft.Framework.Logging.Interfaces": "1.0.0-*",
         "Microsoft.Framework.NotNullAttribute.Internal": { "type": "build", "version": "1.0.0-*" },
         "Microsoft.Framework.OptionsModel": "1.0.0-*"
     },

test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs

@@ -549,13 +549,12 @@ public async Task CanApproveAnyAuthenticatedUser()
                     options.AddPolicy("Any", policy => policy.RequireAuthenticatedUser());
                 });
             });
-            var user = new ClaimsPrincipal(
-                new ClaimsIdentity(
-                    new Claim[] {
-                        new Claim(ClaimTypes.Name, "Name"),
-                    },
-                    "AuthType")
-                );
+            var user = new ClaimsPrincipal(new ClaimsIdentity());
+            user.AddIdentity(new ClaimsIdentity(
+                new Claim[] {
+                    new Claim(ClaimTypes.Name, "Name"),
+                },
+                "AuthType"));
 
             // Act
             var allowed = await authorizationService.AuthorizeAsync(user, null, "Any");