Project

General

Profile

Actions
Copy link

Bug #83822

closed

EXT:form RedirectFinisher breaks HTTP response

Added by Joonas Kauhanen about 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2018-02-09
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

Description

This bug may need certain server environment (gzip, keep-alive, proxy) and browser (Android browser, Opera desktop) to reproduce.
When using the redirect finisher of the new form engine, HTTP page breaks and shows gzip compressed data as garbage on the browser.
I suspect this to be a result of HTTP headers somehow broken after the redirect. When looking at the HTTP response I noticed there is a Location header and Content-Lenght is NOT zero as expected.

Setting $this->response->setHeader('Content-Length', '0'); in the redirectToUri function in RedirectFinisher.php fixed the problem.

Related issues 2 (0 open — 2 closed)

Related to TYPO3 Core - Bug #84495: Confirmation finisher duplicate outputClosed2018-03-20

Actions
Related to TYPO3 Core - Bug #84901: Regression in 8.7.13 for custom form finisherClosedFrank Nägler2018-04-30

Actions
Actions
Copy link
 #1

Updated by Björn Jacob about 6 years ago

  • Sprint Focus set to Remote Sprint
Actions
Copy link
 #2

Updated by Gerrit Code Review about 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/55816

Actions
Copy link
 #3

Updated by Gerrit Code Review about 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/55816

Actions
Copy link
 #4

Updated by Gerrit Code Review about 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/55816

Actions
Copy link
 #5

Updated by Gerrit Code Review about 6 years ago

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/55849

Actions
Copy link
 #6

Updated by Gerrit Code Review about 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/56048

Actions
Copy link
 #7

Updated by Benjamin Franzke about 6 years ago

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

Updated by Ralf Zimmermann about 6 years ago

  • Related to Bug #84495: Confirmation finisher duplicate output added
Actions
Copy link
 #9

Updated by Ralf Zimmermann about 6 years ago

  • Related to Bug #84901: Regression in 8.7.13 for custom form finisher added
Actions
Copy link
 #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF