Ceph » rgw

In fact, I think Yehuda's first comment on the original PR hit the nail on the head. "Why not just take precedence of HTTP_CONTENT_LENGTH over CONTENT_LENGTH". As far as the broken web server argument, they are unlikely to randomly switch between broken and working requests so a simple override configuration setting would be sufficient.

Something like,

    const char* http_content_length = info.env->get("HTTP_CONTENT_LENGTH");

    /* The FastCGI standard says we must use HTTP_CONTENT_LENGTH. If it isn't set
     * we fall back on CONTENT_LENGTH. The rgw content_length_compat config option
     * can be set to force the fallback path in case the web server hands out broken
     * HTTP_CONTENT_LENGTH values */
    if (http_content_length && *http_content_length && !s->cct->conf->rgw_content_length_compat) {
      s->length = http_content_length;
    } else {
      s->length = info.env->get("CONTENT_LENGTH");
    }