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

Include Content-Length response header #151

Closed
wants to merge 1 commit into from
Closed

Include Content-Length response header #151

wants to merge 1 commit into from

Commits on Feb 3, 2024

  1. Include Content-Length response header 

    Motivation:

The 'Content-Length' response header is expected by many clients.  While
it is not possible to specify the content length for on-the-fly
compression, it is possible when sending an individual file.

Modification:

Update DataSelection to optionally return the content length.

Update the 'getData' Response to include the `Content-Length` header
when the request targets a single file that is not placed in a zip
archive.  If the content is dynamically generated (e.g., a zip file)
then no `Content-Length` header is provided.

Result:

IDS now includes the `Content-Length` response header when this is
possible

Signed-off-by: Paul Millar <[email protected]>
    @paulmillar
    paulmillar committed Feb 3, 2024
    Configuration menu
    Copy the full SHA
    7a3a28b View commit details
    Browse the repository at this point in the history