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

Minor updates to CookieAuthenticationHandler and its tests 

1. Remove unnecessary variable declaration.
2. Update test name to a more accurate description.
This commit is contained in:
Troy Dai 2016-08-11 10:15:07 -07:00
parent 4600451dc6
commit 26a7c7016d
2 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/Microsoft.AspNetCore.Authentication.Cookies/CookieAuthenticationHandler.cs
View File

@ -11,7 +11,6 @@ using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Http.Authentication;
using Microsoft.AspNetCore.Http.Features;
using Microsoft.AspNetCore.Http.Features.Authentication;
using Microsoft.Extensions.Primitives;
using Microsoft.Net.Http.Headers;
namespace Microsoft.AspNetCore.Authentication.Cookies
@ -314,8 +313,6 @@ namespace Microsoft.AspNetCore.Authentication.Cookies
if (shouldRedirectToReturnUrl && Response.StatusCode == 200)
{
CookieRedirectContext redirectContext = null;
// set redirect uri in order:
// 1. properties.RedirectUri
// 2. query parameter ReturnUrlParameter
@ -331,8 +328,8 @@ namespace Microsoft.AspNetCore.Authentication.Cookies
if (redirectUri != null)
{
redirectContext = new CookieRedirectContext(Context, Options, redirectUri, properties);
await Options.Events.RedirectToReturnUrl(redirectContext);
await Options.Events.RedirectToReturnUrl(
new CookieRedirectContext(Context, Options, redirectUri, properties));
}
}
}

2
test/Microsoft.AspNetCore.Authentication.Test/Cookies/CookieMiddlewareTests.cs
View File

@ -1100,7 +1100,7 @@ namespace Microsoft.AspNetCore.Authentication.Cookies
}
[Fact]
public async Task AbsoluteRedirectUriIsRejected()
public async Task AbsoluteRedirectUriInQueryStringIsRejected()
{
var options = new CookieAuthenticationOptions
{