Project

General

Profile

Actions

Bug #65719

closed

FE content-length header set too early

Added by Christian Kuhn about 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2015-03-13
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

With 6.2, sending content-length header was enabled by default via config.enableContentLengthHeader considered as "on".

This is done in TSFE->processOutput. Unfortunately, there are a couple of things calculated after this header length is set - and some of those tend to change the output and thus its length afterward. Most prominently the eofe hook. If so, the content length will is set to a wrong value and some webservers or proxies stumble upon that - eg. varnish is a bit picky if header length and actual length differ.

Maybe possible solution: Move the header sending out of processOutput to a later point in TSFE processing - maybe directly before the echo out? That would be more clean and all the "disable if debug and disable if beUserLogin" and all this stuff would be obsolete.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Feature #19732: Enable "Content-length" header by defaultClosed2008-12-22

Actions
Related to TYPO3 Core - Bug #76100: Design problem with sending of header "Content-Length"Closed2016-05-09

Actions
Actions #1

Updated by Christian Kuhn about 9 years ago

  • TYPO3 Version changed from 7 to 6.2
Actions #2

Updated by Tymoteusz Motylewski almost 7 years ago

  • Related to Bug #76100: Design problem with sending of header "Content-Length" added
Actions #3

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54268

Actions #4

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54268

Actions #5

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54268

Actions #6

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54269

Actions #7

Updated by Benni Mack over 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #8

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF