2020-06-30 12:50:28 +02:00
|
|
|
|
include::../description.adoc[]
|
|
|
|
|
|
|
|
|
|
include::../ask-yourself.adoc[]
|
|
|
|
|
|
|
|
|
|
include::../recommended.adoc[]
|
|
|
|
|
|
|
|
|
|
== Sensitive Code Example
|
|
|
|
|
|
2021-01-27 12:06:36 +01:00
|
|
|
|
=== ASP.NET Core MVC
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
[HttpGet]
|
|
|
|
|
public string Get()
|
|
|
|
|
{
|
|
|
|
|
Response.Headers.Add("Access-Control-Allow-Origin", "*"); // Sensitive
|
|
|
|
|
Response.Headers.Add(HeaderNames.AccessControlAllowOrigin, "*"); // Sensitive
|
|
|
|
|
}
|
|
|
|
|
----
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
public void ConfigureServices(IServiceCollection services)
|
|
|
|
|
{
|
|
|
|
|
services.AddCors(options =>
|
|
|
|
|
{
|
|
|
|
|
options.AddDefaultPolicy(builder =>
|
|
|
|
|
{
|
|
|
|
|
builder.WithOrigins("*"); // Sensitive
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
options.AddPolicy(name: "EnableAllPolicy", builder =>
|
|
|
|
|
{
|
|
|
|
|
builder.WithOrigins("*"); // Sensitive
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
options.AddPolicy(name: "OtherPolicy", builder =>
|
|
|
|
|
{
|
|
|
|
|
builder.AllowAnyOrigin(); // Sensitive
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
services.AddControllers();
|
|
|
|
|
}
|
|
|
|
|
----
|
|
|
|
|
|
|
|
|
|
=== ASP.NET MVC
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
public class HomeController : ApiController
|
|
|
|
|
{
|
|
|
|
|
public HttpResponseMessage Get()
|
|
|
|
|
{
|
|
|
|
|
var response = HttpContext.Current.Response;
|
|
|
|
|
|
|
|
|
|
response.Headers.Add("Access-Control-Allow-Origin", "*"); // Sensitive
|
|
|
|
|
response.Headers.Add(HeaderNames.AccessControlAllowOrigin, "*"); // Sensitive
|
|
|
|
|
response.AppendHeader(HeaderNames.AccessControlAllowOrigin, "*"); // Sensitive
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
----
|
|
|
|
|
|
2020-06-30 12:50:28 +02:00
|
|
|
|
----
|
2021-01-27 12:06:36 +01:00
|
|
|
|
[EnableCors(origins: "*", headers: "*", methods: "GET")] // Sensitive
|
|
|
|
|
public HttpResponseMessage Get() => new HttpResponseMessage()
|
|
|
|
|
{
|
|
|
|
|
Content = new StringContent("content")
|
|
|
|
|
};
|
2020-06-30 12:50:28 +02:00
|
|
|
|
----
|
|
|
|
|
|
|
|
|
|
== Compliant Solution
|
|
|
|
|
|
2021-01-27 12:06:36 +01:00
|
|
|
|
=== ASP.NET Core MVC
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
[HttpGet]
|
|
|
|
|
public string Get()
|
|
|
|
|
{
|
|
|
|
|
Response.Headers.Add("Access-Control-Allow-Origin", "https://trustedwebsite.com"); // Safe
|
|
|
|
|
Response.Headers.Add(HeaderNames.AccessControlAllowOrigin, "https://trustedwebsite.com"); // Safe
|
|
|
|
|
}
|
|
|
|
|
{code}
|
|
|
|
|
----
|
|
|
|
|
public void ConfigureServices(IServiceCollection services)
|
|
|
|
|
{
|
|
|
|
|
services.AddCors(options =>
|
|
|
|
|
{
|
|
|
|
|
options.AddDefaultPolicy(builder =>
|
|
|
|
|
{
|
|
|
|
|
builder.WithOrigins("https://trustedwebsite.com", "https://anothertrustedwebsite.com"); // Safe
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
options.AddPolicy(name: "EnableAllPolicy", builder =>
|
|
|
|
|
{
|
|
|
|
|
builder.WithOrigins("https://trustedwebsite.com"); // Safe
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
services.AddControllers();
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
h3. ASP.Net MVC
|
|
|
|
|
----
|
|
|
|
|
public class HomeController : ApiController
|
|
|
|
|
{
|
|
|
|
|
public HttpResponseMessage Get()
|
|
|
|
|
{
|
|
|
|
|
var response = HttpContext.Current.Response;
|
|
|
|
|
|
|
|
|
|
response.Headers.Add("Access-Control-Allow-Origin", "https://trustedwebsite.com");
|
|
|
|
|
response.Headers.Add(HeaderNames.AccessControlAllowOrigin, "https://trustedwebsite.com");
|
|
|
|
|
response.AppendHeader(HeaderNames.AccessControlAllowOrigin, "https://trustedwebsite.com");
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
----
|
|
|
|
|
----
|
2021-01-28 15:53:33 +01:00
|
|
|
|
\[EnableCors(origins: "https://trustedwebsite.com", headers: "*", methods: "GET")]
|
2021-01-27 12:06:36 +01:00
|
|
|
|
public HttpResponseMessage Get() => new HttpResponseMessage()
|
|
|
|
|
{
|
|
|
|
|
Content = new StringContent("content")
|
|
|
|
|
};
|
|
|
|
|
|
2020-06-30 12:50:28 +02:00
|
|
|
|
----
|
|
|
|
|
----
|
|
|
|
|
|
|
|
|
|
include::../see.adoc[]
|