Project

General

Profile

Actions

Bug #82297

closed

Redirect loop with workspace and realurl

Added by Christoph Lehmann about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
Start date:
2017-09-05
Due date:
% Done:

100%

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

Description

We have a multidomain environment:

www.domain.com (main domain and login domain for backend)
www1.domain.com (another site tree with another live domain)

www1.domain is another site tree with its own domain, but we use a subdomain for it to improve cookie handling of backend sessions and backend previews. We also use TCEMAIN.previewdomain extensivly, but that's irreleveant). We also use realurl 2.X.

The current problem is:

An editor works in workspace only in tree of www1.domain.com and does not see the pagetree of www.domain.com. When he opens another tab (FE) with www.domain.com he gets an infinite redirect, which is done in https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php#L2887

        // Prevent redirection loop
        if (!empty($redirectUrl)) {
            // redirect and exit
            HttpUtility::redirect($redirectUrl, HttpUtility::HTTP_STATUS_307);
        }

The block should prevent redirect loops, but does not check if the actual $_SERVER['REQUEST_URI'] is the same as redirectUrl. When i extend the condition to

!empty($redirectUrl) && $redirectUrl !== $_SERVER['REQUEST_URI']

the redirect is not done and the page is shown as expected.

Another ticket, same place in code: https://forge.typo3.org/issues/65447


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #65447: Prevent redirection loop with empty urlClosedNicole Cordes2015-03-02

Actions
Actions #2

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

Actions #3

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

Actions #4

Updated by Benni Mack almost 7 years ago

  • Sprint Focus set to On Location Sprint
Actions #5

Updated by Gerrit Code Review almost 7 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/53977

Actions #6

Updated by Gerrit Code Review almost 7 years ago

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

Actions #7

Updated by Benni Mack almost 7 years ago

  • Related to Bug #65447: Prevent redirection loop with empty url added
Actions #8

Updated by Gerrit Code Review almost 7 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/55590

Actions #9

Updated by Christoph Lehmann almost 7 years ago

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

Updated by Mathias Brodala over 6 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #11

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF