huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Removed redundant if checks (#1390)

This commit is contained in:
Damir Ainullin 2017-08-28 20:12:27 +03:00 committed by Chris Ross
parent 414d596639
commit c8a99e5806
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Microsoft.AspNetCore.Authentication/AuthenticationHandler.cs
View File

@ -142,7 +142,7 @@ namespace Microsoft.AspNetCore.Authentication
Logger.AuthenticationSchemeNotAuthenticated(Scheme.Name);
}
}
else if (result?.Failure != null)
else
{
Logger.AuthenticationSchemeNotAuthenticatedWithFailure(Scheme.Name, result.Failure.Message);
}

4
test/Microsoft.AspNetCore.Authentication.Test/CookieTests.cs
View File

@ -1325,11 +1325,11 @@ namespace Microsoft.AspNetCore.Authentication.Cookies
res.StatusCode = 200;
res.ContentType = "text/xml";
var xml = new XElement("xml");
if (result != null && result?.Ticket?.Principal != null)
if (result?.Ticket?.Principal != null)
{
xml.Add(result.Ticket.Principal.Claims.Select(claim => new XElement("claim", new XAttribute("type", claim.Type), new XAttribute("value", claim.Value))));
}
if (result != null && result?.Ticket?.Properties != null)
if (result?.Ticket?.Properties != null)
{
xml.Add(result.Ticket.Properties.Items.Select(extra => new XElement("extra", new XAttribute("type", extra.Key), new XAttribute("value", extra.Value))));
}