Only look-up Cache-Control once (#23329)
Only look-up the Cache-Control header once, rather than twice if non-empty.
This commit is contained in:
parent
cd9da1c350
commit
c19f3d29ca
|
|
@ -34,11 +34,12 @@ namespace Microsoft.AspNetCore.ResponseCaching
|
||||||
public virtual bool AllowCacheLookup(ResponseCachingContext context)
|
public virtual bool AllowCacheLookup(ResponseCachingContext context)
|
||||||
{
|
{
|
||||||
var requestHeaders = context.HttpContext.Request.Headers;
|
var requestHeaders = context.HttpContext.Request.Headers;
|
||||||
|
var cacheControl = requestHeaders[HeaderNames.CacheControl];
|
||||||
|
|
||||||
// Verify request cache-control parameters
|
// Verify request cache-control parameters
|
||||||
if (!StringValues.IsNullOrEmpty(requestHeaders[HeaderNames.CacheControl]))
|
if (!StringValues.IsNullOrEmpty(cacheControl))
|
||||||
{
|
{
|
||||||
if (HeaderUtilities.ContainsCacheDirective(requestHeaders[HeaderNames.CacheControl], CacheControlHeaderValue.NoCacheString))
|
if (HeaderUtilities.ContainsCacheDirective(cacheControl, CacheControlHeaderValue.NoCacheString))
|
||||||
{
|
{
|
||||||
context.Logger.RequestWithNoCacheNotCacheable();
|
context.Logger.RequestWithNoCacheNotCacheable();
|
||||||
return false;
|
return false;
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue