Bug #88902

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

Added by Georg Großberger almost 2 years ago. Updated over 1 year 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

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
#1

Updated by Georg Ringer almost 2 years ago

  • Status changed from New to Accepted
#2

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

#3

Updated by Georg Großberger almost 2 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.

#4

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

#5

Updated by Gerrit Code Review over 1 year 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

#6

Updated by Daniel Goerz over 1 year ago

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

Updated by Daniel Goerz over 1 year ago

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

Updated by Gerrit Code Review over 1 year 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

#9

Updated by Gerrit Code Review over 1 year 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

#10

Updated by Gerrit Code Review over 1 year 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

#11

Updated by Gerrit Code Review over 1 year 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

#12

Updated by Gerrit Code Review over 1 year 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

#13

Updated by Gerrit Code Review over 1 year 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

#14

Updated by Gerrit Code Review over 1 year 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

#15

Updated by Gerrit Code Review over 1 year 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

#16

Updated by Gerrit Code Review over 1 year 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

#17

Updated by Gerrit Code Review over 1 year 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

#18

Updated by Gerrit Code Review over 1 year 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

#19

Updated by Gerrit Code Review over 1 year 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

#20

Updated by Gerrit Code Review over 1 year 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

#21

Updated by Gerrit Code Review over 1 year 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

#22

Updated by Gerrit Code Review over 1 year 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

#23

Updated by Gerrit Code Review over 1 year 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

#24

Updated by Anonymous over 1 year ago

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

Updated by Joerg Kummer over 1 year ago

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

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed
#27

Updated by Daniel Goerz over 1 year ago

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

Updated by Daniel Goerz 8 months ago

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

Also available in: Atom PDF