#69 Make auth APIs use IEnumerable instead of IList.

Tratcher · Tratcher · commit 10d8b1015e2b · 2014-07-03T09:55:10.000-07:00
diff --git a/src/Microsoft.AspNet.Http/HttpContext.cs b/src/Microsoft.AspNet.Http/HttpContext.cs
@@ -57,14 +57,14 @@ public virtual AuthenticationResult Authenticate(string authenticationType)
             return Authenticate(new[] { authenticationType }).SingleOrDefault();
         }
 
-        public abstract IEnumerable<AuthenticationResult> Authenticate(IList<string> authenticationTypes);
+        public abstract IEnumerable<AuthenticationResult> Authenticate(IEnumerable<string> authenticationTypes);
 
         public virtual async Task<AuthenticationResult> AuthenticateAsync(string authenticationType)
         {
             return (await AuthenticateAsync(new[] { authenticationType })).SingleOrDefault();
         }
 
-        public abstract Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IList<string> authenticationTypes);
+        public abstract Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IEnumerable<string> authenticationTypes);
 
         public virtual Task<WebSocket> AcceptWebSocketAsync()
         {
diff --git a/src/Microsoft.AspNet.Http/HttpResponse.cs b/src/Microsoft.AspNet.Http/HttpResponse.cs
@@ -55,12 +55,12 @@ public virtual void Challenge(string authenticationType, AuthenticationPropertie
             Challenge(new[] { authenticationType }, properties);
         }
 
-        public virtual void Challenge(IList<string> authenticationTypes)
+        public virtual void Challenge(IEnumerable<string> authenticationTypes)
         {
             Challenge(authenticationTypes, properties: null);
         }
 
-        public abstract void Challenge(IList<string> authenticationTypes, AuthenticationProperties properties);
+        public abstract void Challenge(IEnumerable<string> authenticationTypes, AuthenticationProperties properties);
 
         public virtual void SignIn(ClaimsIdentity identity)
         {
@@ -72,12 +72,12 @@ public virtual void SignIn(ClaimsIdentity identity, AuthenticationProperties pro
             SignIn(new[] { identity }, properties);
         }
 
-        public virtual void SignIn(IList<ClaimsIdentity> identities)
+        public virtual void SignIn(IEnumerable<ClaimsIdentity> identities)
         {
             SignIn(identities, properties: null);
         }
 
-        public abstract void SignIn(IList<ClaimsIdentity> identities, AuthenticationProperties properties);
+        public abstract void SignIn(IEnumerable<ClaimsIdentity> identities, AuthenticationProperties properties);
 
         public virtual void SignOut()
         {
@@ -89,6 +89,6 @@ public virtual void SignOut(string authenticationType)
             SignOut(new[] { authenticationType });
         }
 
-        public abstract void SignOut(IList<string> authenticationTypes);
+        public abstract void SignOut(IEnumerable<string> authenticationTypes);
     }
 }
diff --git a/src/Microsoft.AspNet.HttpFeature/Security/IAuthenticateContext.cs b/src/Microsoft.AspNet.HttpFeature/Security/IAuthenticateContext.cs
@@ -10,7 +10,7 @@ namespace Microsoft.AspNet.HttpFeature.Security
     [AssemblyNeutral]
     public interface IAuthenticateContext
     {
-        IList<string> AuthenticationTypes { get; }
+        IEnumerable<string> AuthenticationTypes { get; }
 
         void Authenticated(ClaimsIdentity identity, IDictionary<string, string> properties, IDictionary<string, object> description);
 
diff --git a/src/Microsoft.AspNet.HttpFeature/Security/IChallengeContext.cs b/src/Microsoft.AspNet.HttpFeature/Security/IChallengeContext.cs
@@ -9,7 +9,7 @@ namespace Microsoft.AspNet.HttpFeature.Security
     [AssemblyNeutral]
     public interface IChallengeContext
     {
-        IList<string> AuthenticationTypes {get;}
+        IEnumerable<string> AuthenticationTypes {get;}
         IDictionary<string,string> Properties {get;}
 
         void Accept(string authenticationType, IDictionary<string,object> description);
diff --git a/src/Microsoft.AspNet.HttpFeature/Security/ISignInContext.cs b/src/Microsoft.AspNet.HttpFeature/Security/ISignInContext.cs
@@ -10,7 +10,7 @@ namespace Microsoft.AspNet.HttpFeature.Security
     [AssemblyNeutral]
     public interface ISignInContext
     {
-        IList<ClaimsIdentity> Identities { get; }
+        IEnumerable<ClaimsIdentity> Identities { get; }
         IDictionary<string, string> Properties { get; }
 
         void Accept(string authenticationType, IDictionary<string, object> description);
diff --git a/src/Microsoft.AspNet.HttpFeature/Security/ISignOutContext .cs b/src/Microsoft.AspNet.HttpFeature/Security/ISignOutContext .cs
@@ -9,7 +9,7 @@ namespace Microsoft.AspNet.HttpFeature.Security
     [AssemblyNeutral]
     public interface ISignOutContext 
     {
-        IList<string> AuthenticationTypes { get; }
+        IEnumerable<string> AuthenticationTypes { get; }
 
         void Accept(string authenticationType, IDictionary<string, object> description);
     }
diff --git a/src/Microsoft.AspNet.PipelineCore/DefaultHttpContext.cs b/src/Microsoft.AspNet.PipelineCore/DefaultHttpContext.cs
@@ -178,7 +178,7 @@ public override IEnumerable<AuthenticationDescription> GetAuthenticationTypes()
             return authTypeContext.Results;
         }
 
-        public override IEnumerable<AuthenticationResult> Authenticate(IList<string> authenticationTypes)
+        public override IEnumerable<AuthenticationResult> Authenticate(IEnumerable<string> authenticationTypes)
         {
             if (authenticationTypes == null)
             {
@@ -202,7 +202,7 @@ public override IEnumerable<AuthenticationResult> Authenticate(IList<string> aut
             return authenticateContext.Results;
         }
 
-        public override async Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IList<string> authenticationTypes)
+        public override async Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IEnumerable<string> authenticationTypes)
         {
             if (authenticationTypes == null)
             {
diff --git a/src/Microsoft.AspNet.PipelineCore/DefaultHttpResponse.cs b/src/Microsoft.AspNet.PipelineCore/DefaultHttpResponse.cs
@@ -130,7 +130,7 @@ public override Task WriteAsync(string data)
             return Body.WriteAsync(bytes, 0, bytes.Length);
         }
 
-        public override void Challenge(IList<string> authenticationTypes, AuthenticationProperties properties)
+        public override void Challenge(IEnumerable<string> authenticationTypes, AuthenticationProperties properties)
         {
             if (authenticationTypes == null)
             {
@@ -153,7 +153,7 @@ public override void Challenge(IList<string> authenticationTypes, Authentication
             }
         }
 
-        public override void SignIn(IList<ClaimsIdentity> identities, AuthenticationProperties properties)
+        public override void SignIn(IEnumerable<ClaimsIdentity> identities, AuthenticationProperties properties)
         {
             if (identities == null)
             {
@@ -175,7 +175,7 @@ public override void SignIn(IList<ClaimsIdentity> identities, AuthenticationProp
             }
         }
 
-        public override void SignOut(IList<string> authenticationTypes)
+        public override void SignOut(IEnumerable<string> authenticationTypes)
         {
             if (authenticationTypes == null)
             {
diff --git a/src/Microsoft.AspNet.PipelineCore/Security/AuthTypeContext.cs b/src/Microsoft.AspNet.PipelineCore/Security/AuthTypeContext.cs
@@ -10,16 +10,21 @@ namespace Microsoft.AspNet.PipelineCore.Security
 {
     public class AuthTypeContext : IAuthTypeContext
     {
+        private List<AuthenticationDescription> _results;
+
         public AuthTypeContext()
         {
-            Results = new List<AuthenticationDescription>();
+            _results = new List<AuthenticationDescription>();
         }
 
-        public IList<AuthenticationDescription> Results { get; private set; }
+        public IEnumerable<AuthenticationDescription> Results
+        {
+            get { return _results; }
+        }
                 
         public void Accept(IDictionary<string, object> description)
         {
-            Results.Add(new AuthenticationDescription(description));
+            _results.Add(new AuthenticationDescription(description));
         }
     }
 }
diff --git a/src/Microsoft.AspNet.PipelineCore/Security/AuthenticateContext.cs b/src/Microsoft.AspNet.PipelineCore/Security/AuthenticateContext.cs
@@ -14,33 +14,42 @@ namespace Microsoft.AspNet.PipelineCore.Security
 {
     public class AuthenticateContext : IAuthenticateContext
     {
-        public AuthenticateContext(IList<string> authenticationTypes)
+        private List<AuthenticationResult> _results;
+        private List<string> _accepted;
+
+        public AuthenticateContext(IEnumerable<string> authenticationTypes)
         {
             if (authenticationTypes == null)
             {
                 throw new ArgumentNullException("authenticationType");
             }
             AuthenticationTypes = authenticationTypes;
-            Results = new List<AuthenticationResult>();
-            Accepted = new List<string>();
+            _results = new List<AuthenticationResult>();
+            _accepted = new List<string>();
         }
 
-        public IList<string> AuthenticationTypes { get; private set; }
+        public IEnumerable<string> AuthenticationTypes { get; private set; }
 
-        public IList<AuthenticationResult> Results { get; private set; }
+        public IEnumerable<AuthenticationResult> Results
+        {
+            get { return _results; }
+        }
 
-        public IList<string> Accepted { get; private set; }
+        public IEnumerable<string> Accepted
+        {
+            get { return _accepted; }
+        }
 
         public void Authenticated(ClaimsIdentity identity, IDictionary<string, string> properties, IDictionary<string, object> description)
         {
             var descrip = new AuthenticationDescription(description);
-            Accepted.Add(descrip.AuthenticationType); // may not match identity.AuthType
-            Results.Add(new AuthenticationResult(identity, new AuthenticationProperties(properties), descrip));
+            _accepted.Add(descrip.AuthenticationType); // may not match identity.AuthType
+            _results.Add(new AuthenticationResult(identity, new AuthenticationProperties(properties), descrip));
         }
 
         public void NotAuthenticated(string authenticationType, IDictionary<string, string> properties, IDictionary<string, object> description)
         {
-            Accepted.Add(authenticationType);
+            _accepted.Add(authenticationType);
         }
     }
 }
diff --git a/src/Microsoft.AspNet.PipelineCore/Security/ChallengeContext.cs b/src/Microsoft.AspNet.PipelineCore/Security/ChallengeContext.cs
@@ -12,26 +12,31 @@ namespace Microsoft.AspNet.PipelineCore.Security
 {
     public class ChallengeContext : IChallengeContext
     {
-        public ChallengeContext(IList<string> authenticationTypes, IDictionary<string, string> properties)
+        private List<string> _accepted;
+
+        public ChallengeContext(IEnumerable<string> authenticationTypes, IDictionary<string, string> properties)
         {
             if (authenticationTypes == null)
             {
                 throw new ArgumentNullException();
             }
             AuthenticationTypes = authenticationTypes;
             Properties = properties ?? new Dictionary<string, string>(StringComparer.Ordinal);
-            Accepted = new List<string>();
+            _accepted = new List<string>();
         }
 
-        public IList<string> AuthenticationTypes { get; private set; }
+        public IEnumerable<string> AuthenticationTypes { get; private set; }
 
         public IDictionary<string, string> Properties { get; private set; }
 
-        public IList<string> Accepted { get; private set; }
+        public IEnumerable<string> Accepted
+        {
+            get { return _accepted; }
+        }
         
         public void Accept(string authenticationType, IDictionary<string, object> description)
         {
-            Accepted.Add(authenticationType);
+            _accepted.Add(authenticationType);
         }
     }
 }
diff --git a/src/Microsoft.AspNet.PipelineCore/Security/SignInContext.cs b/src/Microsoft.AspNet.PipelineCore/Security/SignInContext.cs
@@ -10,26 +10,31 @@ namespace Microsoft.AspNet.PipelineCore.Security
 {
     public class SignInContext : ISignInContext
     {
-        public SignInContext(IList<ClaimsIdentity> identities, IDictionary<string, string> dictionary)
+        private List<string> _accepted;
+
+        public SignInContext(IEnumerable<ClaimsIdentity> identities, IDictionary<string, string> dictionary)
         {
             if (identities == null)
             {
                 throw new ArgumentNullException("identities");
             }
             Identities = identities;
             Properties = dictionary ?? new Dictionary<string, string>(StringComparer.Ordinal);
-            Accepted = new List<string>();
+            _accepted = new List<string>();
         }
 
-        public IList<ClaimsIdentity> Identities { get; private set; }
+        public IEnumerable<ClaimsIdentity> Identities { get; private set; }
 
         public IDictionary<string, string> Properties { get; private set; }
 
-        public IList<string> Accepted { get; private set; }
+        public IEnumerable<string> Accepted
+        {
+            get { return _accepted; }
+        }
 
         public void Accept(string authenticationType, IDictionary<string, object> description)
         {
-            Accepted.Add(authenticationType);
+            _accepted.Add(authenticationType);
         }
     }
 }
diff --git a/src/Microsoft.AspNet.PipelineCore/Security/SignOutContext.cs b/src/Microsoft.AspNet.PipelineCore/Security/SignOutContext.cs
@@ -9,23 +9,28 @@ namespace Microsoft.AspNet.PipelineCore.Security
 {
     public class SignOutContext : ISignOutContext
     {
-        public SignOutContext(IList<string> authenticationTypes)
+        private List<string> _accepted;
+
+        public SignOutContext(IEnumerable<string> authenticationTypes)
         {
             if (authenticationTypes == null)
             {
                 throw new ArgumentNullException("authenticationTypes");
             }
             AuthenticationTypes = authenticationTypes;
-            Accepted = new List<string>();
+            _accepted = new List<string>();
         }
 
-        public IList<string> AuthenticationTypes { get; private set; }
+        public IEnumerable<string> AuthenticationTypes { get; private set; }
 
-        public IList<string> Accepted { get; private set; }
+        public IEnumerable<string> Accepted
+        {
+            get { return _accepted; }
+        }
 
         public void Accept(string authenticationType, IDictionary<string, object> description)
         {
-            Accepted.Add(authenticationType);
+            _accepted.Add(authenticationType);
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -57,14 +57,14 @@ public virtual AuthenticationResult Authenticate(string authenticationType)`
`57`	`57`	`return Authenticate(new[] { authenticationType }).SingleOrDefault();`
`58`	`58`	`}`
`59`	`59`
`60`		`- public abstract IEnumerable<AuthenticationResult> Authenticate(IList<string> authenticationTypes);`
	`60`	`+ public abstract IEnumerable<AuthenticationResult> Authenticate(IEnumerable<string> authenticationTypes);`
`61`	`61`
`62`	`62`	`public virtual async Task<AuthenticationResult> AuthenticateAsync(string authenticationType)`
`63`	`63`	`{`
`64`	`64`	`return (await AuthenticateAsync(new[] { authenticationType })).SingleOrDefault();`
`65`	`65`	`}`
`66`	`66`
`67`		`- public abstract Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IList<string> authenticationTypes);`
	`67`	`+ public abstract Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IEnumerable<string> authenticationTypes);`
`68`	`68`
`69`	`69`	`public virtual Task<WebSocket> AcceptWebSocketAsync()`
`70`	`70`	`{`
Original file line number	Diff line number	Diff line change
`@@ -55,12 +55,12 @@ public virtual void Challenge(string authenticationType, AuthenticationPropertie`
`55`	`55`	`Challenge(new[] { authenticationType }, properties);`
`56`	`56`	`}`
`57`	`57`
`58`		`- public virtual void Challenge(IList<string> authenticationTypes)`
	`58`	`+ public virtual void Challenge(IEnumerable<string> authenticationTypes)`
`59`	`59`	`{`
`60`	`60`	`Challenge(authenticationTypes, properties: null);`
`61`	`61`	`}`
`62`	`62`
`63`		`- public abstract void Challenge(IList<string> authenticationTypes, AuthenticationProperties properties);`
	`63`	`+ public abstract void Challenge(IEnumerable<string> authenticationTypes, AuthenticationProperties properties);`
`64`	`64`
`65`	`65`	`public virtual void SignIn(ClaimsIdentity identity)`
`66`	`66`	`{`
`@@ -72,12 +72,12 @@ public virtual void SignIn(ClaimsIdentity identity, AuthenticationProperties pro`
`72`	`72`	`SignIn(new[] { identity }, properties);`
`73`	`73`	`}`
`74`	`74`
`75`		`- public virtual void SignIn(IList<ClaimsIdentity> identities)`
	`75`	`+ public virtual void SignIn(IEnumerable<ClaimsIdentity> identities)`
`76`	`76`	`{`
`77`	`77`	`SignIn(identities, properties: null);`
`78`	`78`	`}`
`79`	`79`
`80`		`- public abstract void SignIn(IList<ClaimsIdentity> identities, AuthenticationProperties properties);`
	`80`	`+ public abstract void SignIn(IEnumerable<ClaimsIdentity> identities, AuthenticationProperties properties);`
`81`	`81`
`82`	`82`	`public virtual void SignOut()`
`83`	`83`	`{`
`@@ -89,6 +89,6 @@ public virtual void SignOut(string authenticationType)`
`89`	`89`	`SignOut(new[] { authenticationType });`
`90`	`90`	`}`
`91`	`91`
`92`		`- public abstract void SignOut(IList<string> authenticationTypes);`
	`92`	`+ public abstract void SignOut(IEnumerable<string> authenticationTypes);`
`93`	`93`	`}`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ namespace Microsoft.AspNet.HttpFeature.Security`
`10`	`10`	`[AssemblyNeutral]`
`11`	`11`	`public interface IAuthenticateContext`
`12`	`12`	`{`
`13`		`- IList<string> AuthenticationTypes { get; }`
	`13`	`+ IEnumerable<string> AuthenticationTypes { get; }`
`14`	`14`
`15`	`15`	`void Authenticated(ClaimsIdentity identity, IDictionary<string, string> properties, IDictionary<string, object> description);`
`16`	`16`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ namespace Microsoft.AspNet.HttpFeature.Security`
`9`	`9`	`[AssemblyNeutral]`
`10`	`10`	`public interface IChallengeContext`
`11`	`11`	`{`
`12`		`- IList<string> AuthenticationTypes {get;}`
	`12`	`+ IEnumerable<string> AuthenticationTypes {get;}`
`13`	`13`	`IDictionary<string,string> Properties {get;}`
`14`	`14`
`15`	`15`	`void Accept(string authenticationType, IDictionary<string,object> description);`
Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ public override IEnumerable<AuthenticationDescription> GetAuthenticationTypes()`
`178`	`178`	`return authTypeContext.Results;`
`179`	`179`	`}`
`180`	`180`
`181`		`- public override IEnumerable<AuthenticationResult> Authenticate(IList<string> authenticationTypes)`
	`181`	`+ public override IEnumerable<AuthenticationResult> Authenticate(IEnumerable<string> authenticationTypes)`
`182`	`182`	`{`
`183`	`183`	`if (authenticationTypes == null)`
`184`	`184`	`{`
`@@ -202,7 +202,7 @@ public override IEnumerable<AuthenticationResult> Authenticate(IList<string> aut`
`202`	`202`	`return authenticateContext.Results;`
`203`	`203`	`}`
`204`	`204`
`205`		`- public override async Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IList<string> authenticationTypes)`
	`205`	`+ public override async Task<IEnumerable<AuthenticationResult>> AuthenticateAsync(IEnumerable<string> authenticationTypes)`
`206`	`206`	`{`
`207`	`207`	`if (authenticationTypes == null)`
`208`	`208`	`{`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ public override Task WriteAsync(string data)`
`130`	`130`	`return Body.WriteAsync(bytes, 0, bytes.Length);`
`131`	`131`	`}`
`132`	`132`
`133`		`- public override void Challenge(IList<string> authenticationTypes, AuthenticationProperties properties)`
	`133`	`+ public override void Challenge(IEnumerable<string> authenticationTypes, AuthenticationProperties properties)`
`134`	`134`	`{`
`135`	`135`	`if (authenticationTypes == null)`
`136`	`136`	`{`
`@@ -153,7 +153,7 @@ public override void Challenge(IList<string> authenticationTypes, Authentication`
`153`	`153`	`}`
`154`	`154`	`}`
`155`	`155`
`156`		`- public override void SignIn(IList<ClaimsIdentity> identities, AuthenticationProperties properties)`
	`156`	`+ public override void SignIn(IEnumerable<ClaimsIdentity> identities, AuthenticationProperties properties)`
`157`	`157`	`{`
`158`	`158`	`if (identities == null)`
`159`	`159`	`{`
`@@ -175,7 +175,7 @@ public override void SignIn(IList<ClaimsIdentity> identities, AuthenticationProp`
`175`	`175`	`}`
`176`	`176`	`}`
`177`	`177`
`178`		`- public override void SignOut(IList<string> authenticationTypes)`
	`178`	`+ public override void SignOut(IEnumerable<string> authenticationTypes)`
`179`	`179`	`{`
`180`	`180`	`if (authenticationTypes == null)`
`181`	`181`	`{`
Original file line number	Diff line number	Diff line change
`@@ -10,16 +10,21 @@ namespace Microsoft.AspNet.PipelineCore.Security`
`10`	`10`	`{`
`11`	`11`	`public class AuthTypeContext : IAuthTypeContext`
`12`	`12`	`{`
	`13`	`+ private List<AuthenticationDescription> _results;`
	`14`	`+`
`13`	`15`	`public AuthTypeContext()`
`14`	`16`	`{`
`15`		`- Results = new List<AuthenticationDescription>();`
	`17`	`+ _results = new List<AuthenticationDescription>();`
`16`	`18`	`}`
`17`	`19`
`18`		`- public IList<AuthenticationDescription> Results { get; private set; }`
	`20`	`+ public IEnumerable<AuthenticationDescription> Results`
	`21`	`+ {`
	`22`	`+ get { return _results; }`
	`23`	`+ }`
`19`	`24`
`20`	`25`	`public void Accept(IDictionary<string, object> description)`
`21`	`26`	`{`
`22`		`- Results.Add(new AuthenticationDescription(description));`
	`27`	`+ _results.Add(new AuthenticationDescription(description));`
`23`	`28`	`}`
`24`	`29`	`}`
`25`	`30`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,33 +14,42 @@ namespace Microsoft.AspNet.PipelineCore.Security`
`14`	`14`	`{`
`15`	`15`	`public class AuthenticateContext : IAuthenticateContext`
`16`	`16`	`{`
`17`		`- public AuthenticateContext(IList<string> authenticationTypes)`
	`17`	`+ private List<AuthenticationResult> _results;`
	`18`	`+ private List<string> _accepted;`
	`19`	`+`
	`20`	`+ public AuthenticateContext(IEnumerable<string> authenticationTypes)`
`18`	`21`	`{`
`19`	`22`	`if (authenticationTypes == null)`
`20`	`23`	`{`
`21`	`24`	`throw new ArgumentNullException("authenticationType");`
`22`	`25`	`}`
`23`	`26`	`AuthenticationTypes = authenticationTypes;`
`24`		`- Results = new List<AuthenticationResult>();`
`25`		`- Accepted = new List<string>();`
	`27`	`+ _results = new List<AuthenticationResult>();`
	`28`	`+ _accepted = new List<string>();`
`26`	`29`	`}`
`27`	`30`
`28`		`- public IList<string> AuthenticationTypes { get; private set; }`
	`31`	`+ public IEnumerable<string> AuthenticationTypes { get; private set; }`
`29`	`32`
`30`		`- public IList<AuthenticationResult> Results { get; private set; }`
	`33`	`+ public IEnumerable<AuthenticationResult> Results`
	`34`	`+ {`
	`35`	`+ get { return _results; }`
	`36`	`+ }`
`31`	`37`
`32`		`- public IList<string> Accepted { get; private set; }`
	`38`	`+ public IEnumerable<string> Accepted`
	`39`	`+ {`
	`40`	`+ get { return _accepted; }`
	`41`	`+ }`
`33`	`42`
`34`	`43`	`public void Authenticated(ClaimsIdentity identity, IDictionary<string, string> properties, IDictionary<string, object> description)`
`35`	`44`	`{`
`36`	`45`	`var descrip = new AuthenticationDescription(description);`
`37`		`- Accepted.Add(descrip.AuthenticationType); // may not match identity.AuthType`
`38`		`- Results.Add(new AuthenticationResult(identity, new AuthenticationProperties(properties), descrip));`
	`46`	`+ _accepted.Add(descrip.AuthenticationType); // may not match identity.AuthType`
	`47`	`+ _results.Add(new AuthenticationResult(identity, new AuthenticationProperties(properties), descrip));`
`39`	`48`	`}`
`40`	`49`
`41`	`50`	`public void NotAuthenticated(string authenticationType, IDictionary<string, string> properties, IDictionary<string, object> description)`
`42`	`51`	`{`
`43`		`- Accepted.Add(authenticationType);`
	`52`	`+ _accepted.Add(authenticationType);`
`44`	`53`	`}`
`45`	`54`	`}`
`46`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,26 +12,31 @@ namespace Microsoft.AspNet.PipelineCore.Security`
`12`	`12`	`{`
`13`	`13`	`public class ChallengeContext : IChallengeContext`
`14`	`14`	`{`
`15`		`- public ChallengeContext(IList<string> authenticationTypes, IDictionary<string, string> properties)`
	`15`	`+ private List<string> _accepted;`
	`16`	`+`
	`17`	`+ public ChallengeContext(IEnumerable<string> authenticationTypes, IDictionary<string, string> properties)`
`16`	`18`	`{`
`17`	`19`	`if (authenticationTypes == null)`
`18`	`20`	`{`
`19`	`21`	`throw new ArgumentNullException();`
`20`	`22`	`}`
`21`	`23`	`AuthenticationTypes = authenticationTypes;`
`22`	`24`	`Properties = properties ?? new Dictionary<string, string>(StringComparer.Ordinal);`
`23`		`- Accepted = new List<string>();`
	`25`	`+ _accepted = new List<string>();`
`24`	`26`	`}`
`25`	`27`
`26`		`- public IList<string> AuthenticationTypes { get; private set; }`
	`28`	`+ public IEnumerable<string> AuthenticationTypes { get; private set; }`
`27`	`29`
`28`	`30`	`public IDictionary<string, string> Properties { get; private set; }`
`29`	`31`
`30`		`- public IList<string> Accepted { get; private set; }`
	`32`	`+ public IEnumerable<string> Accepted`
	`33`	`+ {`
	`34`	`+ get { return _accepted; }`
	`35`	`+ }`
`31`	`36`
`32`	`37`	`public void Accept(string authenticationType, IDictionary<string, object> description)`
`33`	`38`	`{`
`34`		`- Accepted.Add(authenticationType);`
	`39`	`+ _accepted.Add(authenticationType);`
`35`	`40`	`}`
`36`	`41`	`}`
`37`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,26 +10,31 @@ namespace Microsoft.AspNet.PipelineCore.Security`
`10`	`10`	`{`
`11`	`11`	`public class SignInContext : ISignInContext`
`12`	`12`	`{`
`13`		`- public SignInContext(IList<ClaimsIdentity> identities, IDictionary<string, string> dictionary)`
	`13`	`+ private List<string> _accepted;`
	`14`	`+`
	`15`	`+ public SignInContext(IEnumerable<ClaimsIdentity> identities, IDictionary<string, string> dictionary)`
`14`	`16`	`{`
`15`	`17`	`if (identities == null)`
`16`	`18`	`{`
`17`	`19`	`throw new ArgumentNullException("identities");`
`18`	`20`	`}`
`19`	`21`	`Identities = identities;`
`20`	`22`	`Properties = dictionary ?? new Dictionary<string, string>(StringComparer.Ordinal);`
`21`		`- Accepted = new List<string>();`
	`23`	`+ _accepted = new List<string>();`
`22`	`24`	`}`
`23`	`25`
`24`		`- public IList<ClaimsIdentity> Identities { get; private set; }`
	`26`	`+ public IEnumerable<ClaimsIdentity> Identities { get; private set; }`
`25`	`27`
`26`	`28`	`public IDictionary<string, string> Properties { get; private set; }`
`27`	`29`
`28`		`- public IList<string> Accepted { get; private set; }`
	`30`	`+ public IEnumerable<string> Accepted`
	`31`	`+ {`
	`32`	`+ get { return _accepted; }`
	`33`	`+ }`
`29`	`34`
`30`	`35`	`public void Accept(string authenticationType, IDictionary<string, object> description)`
`31`	`36`	`{`
`32`		`- Accepted.Add(authenticationType);`
	`37`	`+ _accepted.Add(authenticationType);`
`33`	`38`	`}`
`34`	`39`	`}`
`35`	`40`	`}`