lsm/Orchard
1
0
Fork 0
mirror of https://github.com/OrchardCMS/Orchard.git synced 2025-11-24 08:46:48 +08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Fixes disposed LifetimeScope issue (#8490)

Browse Source 
* fixed scope in in recomputing the context to figure out whether an antiforgery token needs replacing
This commit is contained in:
Matteo Piovanelli
2021-07-30 08:42:13 +02:00
committed by GitHub
parent b37256fc94
commit 074a0b140d
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/Orchard.Web/Modules/Orchard.OutputCache/Filters/OutputCacheFilter.cs
View File

@@ -236,10 +236,11 @@ namespace Orchard.OutputCache.Filters {
Logger.Debug("Response for item '{0}' will not be cached because status code was set to {1} during rendering.", _cacheKey, response.StatusCode);
return;
}
var cachedOutput = ReplaceRequestVerificationTokenWithBeaconTag(output, response.ContentEncoding);
using (var scope = _workContextAccessor.CreateWorkContextScope()) {
// ask for the WorkContext again so it refreshes in case the scope has been disposed
_workContext = _workContextAccessor.GetContext(filterContext.HttpContext);
var cachedOutput = ReplaceRequestVerificationTokenWithBeaconTag(output, response.ContentEncoding);
var cacheItem = new CacheItem() {
CachedOnUtc = _now,
Duration = cacheDuration,
@@ -648,7 +649,9 @@ namespace Orchard.OutputCache.Filters {
}
private bool PreventCachingRequestVerificationToken() {
return _cacheSettings.CacheAuthenticatedRequests && (!_cacheSettings.VaryByAuthenticationState || _workContext.CurrentUser != null);
return _cacheSettings.CacheAuthenticatedRequests
&& (!_cacheSettings.VaryByAuthenticationState
|| _workContext.CurrentUser != null);
}
protected virtual bool IsIgnoredUrl(string url) {