Bug #52338: Silent configuration generates endless redirect loop - Core - TYPO3 Forge

Core Indexed Search Linkvalidator Release Cycles Team Resources Workspaces & Versioning TYPO3 CMS Usability Team Community Extensions Distributions Feature-Requests TYPO3 6.2 Projects (+)(Archived Projects)

Bug #52338

Task #49162: Rewrite install tool

Silent configuration generates endless redirect loop

Added by Ernesto Baschny almost 2 years ago. Updated almost 2 years ago.

Status:

Resolved

Start date:

Priority:

Must have

Due date:

Assigned To:

% Done:

100%

Category:

Install Tool

Spent time:

Target version:

6.2.0

TYPO3 Version:

6.2

Is Regression:

PHP Version:

Sprint Focus:

Complexity:

Description

If I have some setting in my AdditionalConfiguration like:

$GLOBALS['TYPO3_CONF_VARS']['GFX']['im_v5effects'] = '1';

this will be always overwride whatever I have in my LocalConfiguration. Now the Install Tool has a "Auto-Configuration" which will try to set this setting to "-1" (in LocalConfiguration). And after doing that, it will redirect to itself. But then the AdditionalConfiguration will override this again and the game restarts.

Result is an endless loop.

This has to be detected somehow, because the current behavior is very touchy and you end up in an endless loop very easily.

Related issues

Associated revisions

Revision 263060d4
Added by Anja Leichsenring almost 2 years ago

[BUGFIX] Base silent updates on LocalConfiguration values only

During silent updates the only file that can be influenced is
LocalConfiguration.php. If values are available in
AdditionalConfiguration.php, those take precedence, but can not be
overridden by the updater. This can lead to endless loops, because
preconditions are never met.

So silent update relies now only on values in LocalConfiguration.
For values in AdditionalConfiguration.php the user must take full
responsibility.

Change-Id: I5b450f0cf09f09bd3bb35a519b5b8d6b33e650ac
Resolves: #52338
Releases: 6.2
Reviewed-on: https://review.typo3.org/24140
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Markus Klein
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny

Revision ac3a58b8
Added by Christian Kuhn almost 2 years ago

[TASK] ext:install Break possible redirect loop

The install tool initiates redirects if configuration values were
changed. This is especially done within the "silent upgrade" class.
If something goes wrong there, the system may end up in a redirect
loop and does not show useful debug information.

The patch introduces a simple redirect loop detection and breaks
after ten redirects. This should be enough for the "silent upgrade"
and should be lower than the usual browser based loop detection.

Change-Id: I31164595dc1ada17012525268b57c03499dd2ddf
Resolves: #52414
Related: #52338
Relases: 6.2
Reviewed-on: https://review.typo3.org/24191
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Ernesto Baschny almost 2 years ago

In this case, the SilentConfigurationUpgradeService::setImageMagickDetailSettings is the one that calls for an redirect, because it always end up with a $changedValues.

#2 Updated by Christian Kuhn almost 2 years ago

This probably happens for more values.

Solution is to not access $GLOBALS['TYPO3_CONF_VARS'], but to work with ConfigurationManager getLocalConfigurationValueByPath() and getConfigurationValueByPath()