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

tool_cb_hdr: allow --remote-header-name for all http responses #13303

Closed
wants to merge 1 commit into from
Closed

tool_cb_hdr: allow --remote-header-name for all http responses #13303

wants to merge 1 commit into from

Commits on Apr 6, 2024

  1. tool_cb_hdr: allow --remote-header-name for all http responses 

    - Use the content-disposition filename (server-specified filename) for
  non-2xx responses.

For example, a server may send a content-disposition filename header
with a redirect reply (3xx) but not with the final response (2xx).
Without this change curl would ignore the server's specified filename
and continue to use the filename extracted from the user-specified URL.

This is a partial revert of 75d79a4, which limited content-disposition
and etag header processing to 2xx response codes. There's no explanation
for why that was done.

Reported-by: Morgan Willcock

Fixes curl#13302
Closes #xxxx
    @jay
    jay committed Apr 6, 2024
    Configuration menu
    Copy the full SHA
    fd56bef View commit details
    Browse the repository at this point in the history