React to HttpRequest.IsSecure being renamed
This commit is contained in:
parent
3d6a018793
commit
cf36a601c6
|
|
@ -34,7 +34,7 @@ namespace Microsoft.AspNet.Mvc
|
||||||
|
|
||||||
private void CheckSSLConfig(HttpContext httpContext)
|
private void CheckSSLConfig(HttpContext httpContext)
|
||||||
{
|
{
|
||||||
if (_config.RequireSSL && !httpContext.Request.IsSecure)
|
if (_config.RequireSSL && !httpContext.Request.IsHttps)
|
||||||
{
|
{
|
||||||
throw new InvalidOperationException(Resources.AntiForgeryWorker_RequireSSL);
|
throw new InvalidOperationException(Resources.AntiForgeryWorker_RequireSSL);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -14,7 +14,7 @@ namespace Microsoft.AspNet.Mvc
|
||||||
|
|
||||||
public virtual void OnAuthorization([NotNull]AuthorizationContext filterContext)
|
public virtual void OnAuthorization([NotNull]AuthorizationContext filterContext)
|
||||||
{
|
{
|
||||||
if (!filterContext.HttpContext.Request.IsSecure)
|
if (!filterContext.HttpContext.Request.IsHttps)
|
||||||
{
|
{
|
||||||
HandleNonHttpsRequest(filterContext);
|
HandleNonHttpsRequest(filterContext);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -21,7 +21,7 @@ namespace Microsoft.AspNet.Mvc.Core.Test
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
var mockHttpContext = new Mock<HttpContext>();
|
var mockHttpContext = new Mock<HttpContext>();
|
||||||
mockHttpContext.Setup(o => o.Request.IsSecure)
|
mockHttpContext.Setup(o => o.Request.IsHttps)
|
||||||
.Returns(false);
|
.Returns(false);
|
||||||
|
|
||||||
var config = new AntiForgeryOptions()
|
var config = new AntiForgeryOptions()
|
||||||
|
|
@ -52,7 +52,7 @@ namespace Microsoft.AspNet.Mvc.Core.Test
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
var mockHttpContext = new Mock<HttpContext>();
|
var mockHttpContext = new Mock<HttpContext>();
|
||||||
mockHttpContext.Setup(o => o.Request.IsSecure)
|
mockHttpContext.Setup(o => o.Request.IsHttps)
|
||||||
.Returns(false);
|
.Returns(false);
|
||||||
|
|
||||||
var config = new AntiForgeryOptions()
|
var config = new AntiForgeryOptions()
|
||||||
|
|
@ -81,7 +81,7 @@ namespace Microsoft.AspNet.Mvc.Core.Test
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
var mockHttpContext = new Mock<HttpContext>();
|
var mockHttpContext = new Mock<HttpContext>();
|
||||||
mockHttpContext.Setup(o => o.Request.IsSecure)
|
mockHttpContext.Setup(o => o.Request.IsHttps)
|
||||||
.Returns(false);
|
.Returns(false);
|
||||||
|
|
||||||
var config = new AntiForgeryOptions()
|
var config = new AntiForgeryOptions()
|
||||||
|
|
@ -109,7 +109,7 @@ namespace Microsoft.AspNet.Mvc.Core.Test
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
var mockHttpContext = new Mock<HttpContext>();
|
var mockHttpContext = new Mock<HttpContext>();
|
||||||
mockHttpContext.Setup(o => o.Request.IsSecure)
|
mockHttpContext.Setup(o => o.Request.IsHttps)
|
||||||
.Returns(false);
|
.Returns(false);
|
||||||
|
|
||||||
var config = new AntiForgeryOptions()
|
var config = new AntiForgeryOptions()
|
||||||
|
|
|
||||||
|
|
@ -15,7 +15,7 @@ namespace Microsoft.AspNet.Mvc
|
||||||
public class RequireHttpsAttributeTests
|
public class RequireHttpsAttributeTests
|
||||||
{
|
{
|
||||||
[Fact]
|
[Fact]
|
||||||
public void OnAuthorization_AllowsTheRequestIfItIsSecure()
|
public void OnAuthorization_AllowsTheRequestIfItIsHttps()
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
var requestContext = new DefaultHttpContext();
|
var requestContext = new DefaultHttpContext();
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue