You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Eric Covener <co...@gmail.com> on 2011/09/07 19:43:39 UTC

Re: svn commit: r1166282 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/core.xml include/http_core.h modules/http/byterange_filter.c server/core.c

On Wed, Sep 7, 2011 at 1:29 PM,  <co...@apache.org> wrote:
> Author: covener
> Date: Wed Sep  7 17:29:49 2011
> New Revision: 1166282
>
> URL: http://svn.apache.org/viewvc?rev=1166282&view=rev
> Log:
> take care of some MaxRanges feedback:
>  * allow "none" to be expressed in config
>  * send Accept-Ranges: none with MaxRanges none
>  * stop accepting confusing/ambiguous "0", start accepting "unlimited".
> @@ -4196,7 +4211,9 @@ static int default_handler(request_rec *
>         ap_update_mtime(r, r->finfo.mtime);
>         ap_set_last_modified(r);
>         ap_set_etag(r);
> -        apr_table_setn(r->headers_out, "Accept-Ranges", "bytes");
> +        apr_table_setn(r->headers_out, "Accept-Ranges",
> +                       (d->max_ranges == AP_MAXRANGES_NORANGES) ? "none"
> +                                                                : "bytes");
>         ap_set_content_length(r, r->finfo.size);
>         if (bld_content_md5) {
>             apr_table_setn(r->headers_out, "Content-MD5",
>

Should have committed this bit separately and not buried it in this
rev.  Highlighting here.


-- 
Eric Covener
covener@gmail.com