Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

range: New http_GetContentRange() semantics #4091

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

range: New http_GetContentRange() semantics #4091

wants to merge 2 commits into from

Commits on Mar 27, 2024

  1. range: New http_GetContentRange() semantics 

    This function returns the content length and puts range indices in
lo and hi arguments. A content length of -1 used to be interpreted
as something to ignore, but there is a case where the content length
may be unknown.

Since we can't represent a zero-length range, because both bounds are
inclusive, zero now denotes the lack of content-range header.

Unknown range units are treated as errors as they wouldn't pass the
busyobj check for the range header, even for pass transactions. The
only way to use "extension" range units is to turn http_range_support
off.

The calling convention for http_GetContentRange() remains otherwise
the same, and for good measure http_GetContentLength() received a
similar description.

Fixes varnishcache#4089
    @dridi
    dridi committed Mar 27, 2024
    Configuration menu
    Copy the full SHA
    788227e View commit details
    Browse the repository at this point in the history

  2. vrt: Register http_GetContentRange() change

    @dridi
    dridi committed Mar 27, 2024
    Configuration menu
    Copy the full SHA
    e400094 View commit details
    Browse the repository at this point in the history