Skip to content
This repository was archived by the owner on Nov 20, 2018. It is now read-only.

Remove AuthenticateResult.Handle()/Handled #871

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ namespace Microsoft.AspNetCore.Authentication
/// </summary>
public class AuthenticateResult
{
private AuthenticateResult() { }
protected AuthenticateResult() { }

/// <summary>
/// If a ticket was produced, authenticate was successful.
Expand All @@ -21,7 +21,7 @@ private AuthenticateResult() { }
/// <summary>
/// The authentication ticket.
/// </summary>
public AuthenticationTicket Ticket { get; private set; }
public AuthenticationTicket Ticket { get; protected set; }

/// <summary>
/// Gets the claims-principal with authenticated user identities.
Expand All @@ -36,18 +36,12 @@ private AuthenticateResult() { }
/// <summary>
/// Holds failure information from the authentication.
/// </summary>
public Exception Failure { get; private set; }

/// <summary>
/// Indicates that stage of authentication was directly handled by user intervention and no
/// further processing should be attempted.
/// </summary>
public bool Handled { get; private set; }
public Exception Failure { get; protected set; }

/// <summary>
/// Indicates that there was no information returned for this authentication scheme.
/// </summary>
public bool Nothing { get; private set; }
public bool Nothing { get; protected set; }

/// <summary>
/// Indicates that authentication was successful.
Expand All @@ -63,16 +57,6 @@ public static AuthenticateResult Success(AuthenticationTicket ticket)
return new AuthenticateResult() { Ticket = ticket };
}

/// <summary>
/// Indicates that stage of authentication was directly handled by user intervention and no
/// further processing should be attempted.
/// </summary>
/// <returns>The result.</returns>
public static AuthenticateResult Handle()
{
return new AuthenticateResult() { Handled = true };
}

/// <summary>
/// Indicates that there was no information returned for this authentication scheme.
/// </summary>
Expand Down