Project

General

Profile

Actions

Bug #88902

closed

Redirects are not resolved if there is a base path in default site

Added by Georg Großberger over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2019-08-03
Due date:
% Done:

100%

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

Description

If a site has a path in its base URL, any redirect, not containing this base path, will never be hit, because the redirect to the base is done before.

How to reproduce:

1. Have a site with a path in its base URL, eg.: if the default language is EN, set its base to "/en"

2. Add a redirect which does not start with "/en", eg.: "/about" -> "/en/company/about-us"

Affects: master (v10) and v9


Related issues 5 (1 open4 closed)

Related to TYPO3 Core - Bug #88543: Static routes does not work if no language serves root levelClosed2019-06-12

Actions
Related to TYPO3 Core - Bug #86835: Static route sitemap.xml is still not working on single-page treeNew2018-11-02

Actions
Related to TYPO3 Core - Bug #89719: Rename Changelog file Important-88902 to FeatureClosed2019-11-21

Actions
Related to TYPO3 Core - Task #92352: Remove feature switch for rearanged redirect middlewares and set new defaultClosedDaniel Goerz2020-09-21

Actions
Has duplicate TYPO3 Core - Bug #87455: Redirects for root path on Multi-Language pageClosed2019-01-16

Actions
Actions #1

Updated by Georg Ringer over 4 years ago

  • Status changed from New to Accepted
Actions #2

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted 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/c/Packages/TYPO3.CMS/+/61426

Actions #3

Updated by Georg Großberger over 4 years ago

I'm unsure about the patch, because it puts the redirects even before the static routes. This can be good, but may be not in other cases. Would need definition, which is more important for TYPO3.

Actions #4

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/61426

Actions #5

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/61426

Actions #6

Updated by Daniel Goerz over 4 years ago

  • Related to Bug #88543: Static routes does not work if no language serves root level added
Actions #7

Updated by Daniel Goerz over 4 years ago

  • Related to Bug #86835: Static route sitemap.xml is still not working on single-page tree added
Actions #8

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/61426

Actions #9

Updated by Gerrit Code Review over 4 years ago

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

Actions #10

Updated by Gerrit Code Review over 4 years ago

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

Actions #11

Updated by Gerrit Code Review over 4 years ago

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

Actions #12

Updated by Gerrit Code Review over 4 years ago

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

Actions #13

Updated by Gerrit Code Review over 4 years ago

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

Actions #14

Updated by Gerrit Code Review over 4 years ago

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

Actions #15

Updated by Gerrit Code Review over 4 years ago

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

Actions #16

Updated by Gerrit Code Review over 4 years ago

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

Actions #17

Updated by Gerrit Code Review over 4 years ago

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

Actions #18

Updated by Gerrit Code Review over 4 years ago

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

Actions #19

Updated by Gerrit Code Review over 4 years ago

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

Actions #20

Updated by Gerrit Code Review over 4 years ago

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

Actions #21

Updated by Gerrit Code Review over 4 years ago

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

Actions #22

Updated by Gerrit Code Review over 4 years ago

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

Actions #23

Updated by Gerrit Code Review over 4 years ago

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

Actions #24

Updated by Anonymous over 4 years ago

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

Updated by Joerg Kummer over 4 years ago

  • Related to Bug #89719: Rename Changelog file Important-88902 to Feature added
Actions #26

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions #27

Updated by Daniel Goerz over 4 years ago

  • Has duplicate Bug #87455: Redirects for root path on Multi-Language page added
Actions #28

Updated by Daniel Goerz over 3 years ago

  • Related to Task #92352: Remove feature switch for rearanged redirect middlewares and set new default added
Actions

Also available in: Atom PDF