Quoting Graham Leggett <[EMAIL PROTECTED]>: > This looks quite cool - can you post more details?
Will post more info this weekend. > > > During our testing phase we noticed a problem with proxy_cache.c, line > 1157. It > > seems that Cache-Control: max-age=0, or Cache-Control: of any kind in the > > request would not have been fully functional (at minimum) without this > change. > > > > /* extract max-age from request */ > > if (cc_cresp && ap_proxy_liststr(cc_req, "max-age", &val)) > > Good catch! This code would have still worked, but the test was for > whether cc_req was non-NULL to prevent a potential segfault, and we were > testing the wrong variable. > > Will patch. Thanks! However the code didn't exactly work. If the cached response didn't have the "cache-control" defined (cc_resp = NULL), then the cc_req wouldn't be processed :(. Geff
