Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

content-length: when we don't preserve headers, clear the CL header #702

Closed
wants to merge 1 commit into from

Conversation

oschaaf
Copy link
Member

@oschaaf oschaaf commented May 13, 2014

I haven't been able to reproduce a problem with this, looking at the code it seems to me that we do need to do this.

@jeffkaufman
Copy link
Contributor

I wonder if we need both this and r3996.

@oschaaf
Copy link
Member Author

oschaaf commented May 15, 2014

@jeffkaufman "I wonder if we need both this and r3996."
Probably not to address the initial problem that inspired this.
But: do we ever want to allow an original content-length header to be forwarded at the point where this change clears it? If the answer is yes, this change doesn't make sense.

@oschaaf
Copy link
Member Author

oschaaf commented Feb 13, 2015

This probably is outdated - closing

@oschaaf oschaaf closed this Feb 13, 2015
@oschaaf oschaaf deleted the oschaaf-1.7.30.4-content-length branch February 13, 2015 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants