#273 - Use POCOs for auth context objects.

Author: Tratcher

src/Microsoft.AspNet.Http.Interfaces/Authentication/AuthenticateContext.cs

@@ -0,0 +1,40 @@
+// 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.Collections.Generic;
+using System.Security.Claims;
+using Microsoft.Framework.Internal;
+
+namespace Microsoft.AspNet.Http.Authentication
+{
+    public class AuthenticateContext
+    {
+        public AuthenticateContext([NotNull] string authenticationScheme)
+        {
+            AuthenticationScheme = authenticationScheme;
+        }
+
+        public string AuthenticationScheme { get; }
+
+        public bool Accepted { get; private set; }
+
+        public ClaimsPrincipal Principal { get; private set; }
+
+        public IDictionary<string, string> Properties { get; private set; }
+
+        public IDictionary<string, object> Description { get; private set; }
+
+        public virtual void Authenticated(ClaimsPrincipal principal, IDictionary<string, string> properties, IDictionary<string, object> description)
+        {
+            Accepted = true;
+            Principal = principal;
+            Properties = properties;
+            Description = description;
+        }
+
+        public virtual void NotAuthenticated()
+        {
+            Accepted = true;
+        }
+    }
+}

src/Microsoft.AspNet.Http/Authentication/ChallengeContext.cs renamed to src/Microsoft.AspNet.Http.Interfaces/Authentication/ChallengeContext.cs

@@ -3,35 +3,26 @@
 
 using System;
 using System.Collections.Generic;
-using Microsoft.Framework.Internal;
 
 namespace Microsoft.AspNet.Http.Authentication
 {
-    public class ChallengeContext : IChallengeContext
+    public class ChallengeContext
     {
-        private bool _accepted;
-
         public ChallengeContext(string authenticationScheme, IDictionary<string, string> properties)
         {
             AuthenticationScheme = authenticationScheme;
             Properties = properties ?? new Dictionary<string, string>(StringComparer.Ordinal);
-
-            // The default Challenge with no scheme is always accepted
-            _accepted = string.IsNullOrEmpty(authenticationScheme);
         }
 
-        public string AuthenticationScheme { get; private set; }
+        public string AuthenticationScheme { get; }
 
-        public IDictionary<string, string> Properties { get; private set; }
+        public IDictionary<string, string> Properties { get; }
 
-        public bool Accepted
-        {
-            get { return _accepted; }
-        }
+        public bool Accepted { get; private set; }
 
         public void Accept()
         {
-            _accepted = true;
+            Accepted = true;
         }
     }
-}
+}

src/Microsoft.AspNet.Http/Authentication/DescribeSchemesContext.cs renamed to src/Microsoft.AspNet.Http.Interfaces/Authentication/DescribeSchemesContext.cs

@@ -5,23 +5,23 @@
 
 namespace Microsoft.AspNet.Http.Authentication
 {
-    public class DescribeSchemesContext : IDescribeSchemesContext
+    public class DescribeSchemesContext
     {
-        private List<AuthenticationDescription> _results;
+        private List<IDictionary<string, object>> _results;
 
         public DescribeSchemesContext()
         {
-            _results = new List<AuthenticationDescription>();
+            _results = new List<IDictionary<string, object>>();
         }
 
-        public IEnumerable<AuthenticationDescription> Results
+        public IEnumerable<IDictionary<string, object>> Results
         {
             get { return _results; }
         }
 
         public void Accept(IDictionary<string, object> description)
         {
-            _results.Add(new AuthenticationDescription(description));
+            _results.Add(description);
         }
     }
-}
+}

src/Microsoft.AspNet.Http.Interfaces/Authentication/IAuthenticateContext.cs

@@ -7,13 +7,16 @@ namespace Microsoft.AspNet.Http.Authentication
 {
     public interface IAuthenticationHandler
     {
-        void GetDescriptions(IDescribeSchemesContext context);
+        void GetDescriptions(DescribeSchemesContext context);
 
-        void Authenticate(IAuthenticateContext context);
-        Task AuthenticateAsync(IAuthenticateContext context);
+        void Authenticate(AuthenticateContext context);
 
-        void Challenge(IChallengeContext context);
-        void SignIn(ISignInContext context);
-        void SignOut(ISignOutContext context);
+        Task AuthenticateAsync(AuthenticateContext context);
+
+        void Challenge(ChallengeContext context);
+
+        void SignIn(SignInContext context);
+
+        void SignOut(SignOutContext context);
     }
 }

src/Microsoft.AspNet.Http.Interfaces/Authentication/IAuthenticationHandler.cs

@@ -8,31 +8,26 @@
 
 namespace Microsoft.AspNet.Http.Authentication
 {
-    public class SignInContext : ISignInContext
+    public class SignInContext
     {
-        private bool _accepted;
-
-        public SignInContext([NotNull] string authenticationScheme, [NotNull] ClaimsPrincipal principal, IDictionary<string, string> dictionary)
+        public SignInContext([NotNull] string authenticationScheme, [NotNull] ClaimsPrincipal principal, IDictionary<string, string> properties)
         {
             AuthenticationScheme = authenticationScheme;
             Principal = principal;
-            Properties = dictionary ?? new Dictionary<string, string>(StringComparer.Ordinal);
+            Properties = properties ?? new Dictionary<string, string>(StringComparer.Ordinal);
         }
 
+        public string AuthenticationScheme { get; }
+
         public ClaimsPrincipal Principal { get; }
 
         public IDictionary<string, string> Properties { get; }
 
-        public string AuthenticationScheme { get; }
-
-        public bool Accepted
-        {
-            get { return _accepted; }
-        }
+        public bool Accepted { get; private set; }
 
-        public void Accept(IDictionary<string, object> description)
+        public void Accept()
         {
-            _accepted = true;
+            Accepted = true;
         }
     }
 }

src/Microsoft.AspNet.Http.Interfaces/Authentication/IChallengeContext.cs

@@ -7,10 +7,8 @@
 
 namespace Microsoft.AspNet.Http.Authentication
 {
-    public class SignOutContext : ISignOutContext
+    public class SignOutContext
     {
-        private bool _accepted;
-
         public SignOutContext([NotNull] string authenticationScheme, IDictionary<string, string> properties)
         {
             AuthenticationScheme = authenticationScheme;
@@ -21,14 +19,11 @@ public SignOutContext([NotNull] string authenticationScheme, IDictionary<string,
 
         public IDictionary<string, string> Properties { get; }
 
-        public bool Accepted
-        {
-            get { return _accepted; }
-        }
+        public bool Accepted { get; private set; }
 
         public void Accept()
         {
-            _accepted = true;
+            Accepted = true;
         }
     }
-}
+}

src/Microsoft.AspNet.Http.Interfaces/Authentication/IDescribeSchemesContext.cs

@@ -1,16 +1,21 @@
 {
-  "version": "1.0.0-*",
-  "description": "ASP.NET 5 HTTP feature interface definitions.",
-  "frameworks": {
-    "dnx451": {},
-    "dnxcore50": {
-      "dependencies": {
-        "System.Net.Primitives": "4.0.10-beta-*",
-        "System.Net.WebSockets" : "4.0.0-beta-*",
-        "System.Security.Claims": "4.0.0-beta-*",
-        "System.Security.Cryptography.X509Certificates": "4.0.0-beta-*",
-        "System.Security.Principal": "4.0.0-beta-*"
-      }
+    "version": "1.0.0-*",
+    "description": "ASP.NET 5 HTTP feature interface definitions.",
+    "dependencies": {
+        "Microsoft.Framework.NotNullAttribute.Internal": { "type": "build", "version": "1.0.0-*" }
+    },
+    "frameworks": {
+        "dnx451": { },
+        "dnxcore50": {
+            "dependencies": {
+                "System.Collections": "4.0.10-beta-*",
+                "System.Net.Primitives": "4.0.10-beta-*",
+                "System.Net.WebSockets": "4.0.0-beta-*",
+                "System.Runtime.Extensions": "4.0.10-beta-*",
+                "System.Security.Claims": "4.0.0-beta-*",
+                "System.Security.Cryptography.X509Certificates": "4.0.0-beta-*",
+                "System.Security.Principal": "4.0.0-beta-*"
+            }
+        }
     }
-  }
 }