nginx proxy_cache 缓存不住

问题描述

最近在使用proxy_cache的时候发现一个缓存不住的问题，比较隐蔽，现在抛出来，看看大家有遇到相同的情况没。

触发场景

响应头中包含Expires 和 Cache-Control两个头，但是Expires是一个过期的时间(比当前时间小)。

RFC中规定 cache-control：max-age的优先级高于Expires，看起来没有问题，理论上没有问题，但是nginx处理这块的代码有点问题。会导致资源不缓存(Expires头先收到，就会先于Cache-Control处理，在Expires非法后，会将cachable标置位0，所以cache-control就不会生效)。

代码分析

先看一下expires头和cache-control头的处理代码

static ngx_int_t
ngx_http_upstream_process_expires(ngx_http_request_t *r, ngx_table_elt_t *h,
    ngx_uint_t offset)
{

    //.....
    //判断之前有没有处理过cache-control或其他优先级高的缓存头
    if (r->cache->valid_sec != 0) {
        return NGX_OK;
    }

    expires = ngx_parse_http_time(h->value.data, h->value.len);

    //如果Expires时间异常，会把cacheable这个标置为不可用
    if (expires == NGX_ERROR || expires < ngx_time()) { 
        u->cacheable = 0;
        return NGX_OK;
    }
    //.....
}


static ngx_int_t
ngx_http_upstream_process_cache_control(ngx_http_request_t *r,
    ngx_table_elt_t *h, ngx_uint_t offset)
{
    //....


    if (p == NULL) {
        p = ngx_strlcasestrn(start, last, (u_char *) "max-age=", 8 - 1);
        offset = 8;
    }

    if (p) {
        n = 0;

        for (p += offset; p < last; p++) {
            if (*p == ',' || *p == ';' || *p == ' ') {
                break;
            }

            if (*p >= '0' && *p <= '9') {
                n = n * 10 + (*p - '0');
                continue;
            }

            u->cacheable = 0;
            return NGX_OK;
        }

        if (n == 0) {
            u->cacheable = 0;
            return NGX_OK;
        }
        //解析成功的时候，就只更新了缓存时长，没有将cacheable的标志位置为1
        r->cache->valid_sec = ngx_time() + n;

    //....
}

所以如果nginx收到的响应头中 Expires比cache-control先收到的化，就会触发这个bug

解决方案

在nginx中添加判断，如果有cache-control头，就将Expires 隐掉。