<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">I noticed there was no test suite for this new feature. I’m not expecting my patch to be approved as time-based tests slow down test suites - but I think it warrants some testing.</div><div class=""><br class=""></div><div class="">I have a conflicting requirement that isn’t easily supported by nginx currently; I want to enable ‘proxy_cache_use_stale updating’ for responses that do not include stale-while-revalidate. Unfortunately using this directive breaks the stale-while-revalidate functionality. See the tests patch for an example. Removing the 'proxy_cache_use_stale updating’ directive makes the tests pass as expected.</div><div class=""><br class=""></div><div class="">Should ‘updating’ support both cases, or should a new directive be added to handle this case? AIUI, stale-while-revalidate is an explicit instruction from upstream not to serve the content after the stale period expires. Without that directive, we are allowed to serve stale for an indefinite amount of time (subject to exceptions in an HTTP request per RFC 2616…).</div><br class=""><div><blockquote type="cite" class=""><div class="">On 10 Feb 2017, at 17:30, <a href="mailto:nginx-devel-request@nginx.org" class="">nginx-devel-request@nginx.org</a> wrote:</div><br class="Apple-interchange-newline"><div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">[nginx] Cache: support for stale-while-revalidate and</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span class="Apple-tab-span" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: pre; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"> </span><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">stale-if-error.</span></div></blockquote></div><br class=""></body></html>