Bug #53931

ActionRequest should not serialize parent HTTP request

Added by Sebastian Kurfuerst over 1 year ago. Updated about 1 year ago.

Status:Resolved Start date:2013-11-25
Priority:Should have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%

Category:MVC
Target version:TYPO3 Flow Base Distribution - 2.0.1
PHP Version:5.3 Complexity:
Has patch:Yes Affected Flow version:Flow 2.0.0

Description

When working with WebRedirects and authentication, the intercepted
request is serialized with the security context. While it makes sense
to do so, we should not include the reference to the base HTTP request
in this case, as this includes lots of cruft like $_SERVER etc.


Related issues

related to TYPO3.Flow - Bug #49289: Serialization error on WebRedirect with PHP 5.3 Resolved 2013-06-20

Associated revisions

Revision 6ec233ef
Added by Sebastian Kurfuerst over 1 year ago

[BUGFIX] ActionRequest should not serialize parent HTTP request

When working with WebRedirects and authentication, the intercepted
request is serialized with the security context. While it makes sense
to do so, we should not include the reference to the base HTTP request
in this case, as this includes lots of cruft like $_SERVER etc.

In order to test this, just look at the serialized session files
when a web redirect is taking place.

Change-Id: Ie8f27af4446711f1573da6ecfc3b00db80509c5c
Releases: master, 2.0
Resolves: #53931

Revision 896a47d5
Added by Sebastian Kurfuerst over 1 year ago

[BUGFIX] ActionRequest should not serialize parent HTTP request

When working with WebRedirects and authentication, the intercepted
request is serialized with the security context. While it makes sense
to do so, we should not include the reference to the base HTTP request
in this case, as this includes lots of cruft like $_SERVER etc.

In order to test this, just look at the serialized session files
when a web redirect is taking place.

Change-Id: Ie8f27af4446711f1573da6ecfc3b00db80509c5c
Releases: master, 2.0
Resolves: #53931

History

#1 Updated by Sebastian Kurfuerst over 1 year ago

  • Category set to MVC
  • Assigned To set to Sebastian Kurfuerst
  • Target version set to 2.0.1
  • PHP Version set to 5.3
  • Has patch changed from No to Yes

#2 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted to Under Review

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

#3 Updated by Gerrit Code Review over 1 year ago

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

#4 Updated by Sebastian Kurfuerst about 1 year ago

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

Also available in: Atom PDF