Project

General

Profile

Actions

Bug #89090

closed

No redirect loop prevention for slugs / redirects

Added by Sybille Peters 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-09-05
Due date:
% Done:

100%

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

Description

System

TYPO3 version 9.5.9

Possible Scenario 1 (realistic)

  1. Slug for page 15 is /something/other
  2. A redirect gets created to this page: /other -> t3://page?uid=15
  3. Now, the slug is changed fpr page 15 to /other
  4. The page redirects to itself (redirect loop). Not possible to access page at all

Possible scenario 2 (hypothetical, but may happen)

It is possible to enter redirects to the same page in the redirects module:

  1. Slug for page 15 is /other
  2. A redirect is entered: /other to t3://page?uid=79
  3. page /other is not accessible anymore because of redirect loop

Related issues 2 (1 open1 closed)

Related to TYPO3 Core - Feature #89115: Auto-create Redirects on Slug ChangesClosedFrank Nägler2019-09-09

Actions
Related to TYPO3 Core - Bug #92748: Infinite redirects (redirect loop) when updating slugsNew2020-11-02

Actions
Actions #1

Updated by Georg Ringer over 4 years ago

There are no redirects created after slug change...

Actions #2

Updated by Sybille Peters over 4 years ago

The scenarios are about creating redirects manually.

Actions #3

Updated by Gerrit Code Review over 4 years ago

  • Status changed from New to Under Review

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/+/61627

Actions #4

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/+/61627

Actions #5

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/+/61627

Actions #6

Updated by Frank Nägler over 4 years ago

Georg Ringer wrote:

There are no redirects created after slug change...

not yet... wait some days ;)

Actions #7

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/+/61627

Actions #8

Updated by Sybille Peters over 4 years ago

Frank, it would be interesting to know where this is going conceptually. If you can point me to information, that would be helpful (and also prevent other people from creating pointless issues, patches and extensions which "fix" the URL handling). Thanks.

There are some open questions / issues which might already be resolved, if I had a better idea of where the slug / redirects handling is going in TYPO3.

E.g. automatically creating redirects on slug changes.

In my experience, people do tend to change URLs and the tables will likely be bloated over time. So a good expiration and cleanup strategy would be useful. If someone changes the slug several times in a short period of time, you may want to replace the redirect and not create new ones. (For example someone chooses a slug which already existed, a "/something-1" gets created, gets changed again, etc.).

Currently, there is no possibility for editors to see existing slugs and redirects for a page and subpages. (in realurl this was possible) This results in people fumbling in the dark and possibly making bad choices for the slugs.

Also, the slug no longer depends on slug of parent, so long term the slugs will likely generally diverge. I heard something may be implemented, where you can choose to change slugs of subpages as well.

Also, I am not sure how to handle the permissions. You have to give editors permissions to the slugs so they can change them after page creation, but afterwards you may only want a few trusted users to be able to change slugs and access redirects. It might be useful to be able to protect the slug field for a page.

(I skimmed through the documentation and it is good, but does not answer all questions)

Actions #9

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/+/61627

Actions #10

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/+/61627

Actions #11

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/+/61627

Actions #12

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/+/61627

Actions #13

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/+/61627

Actions #14

Updated by Andreas Fernandez over 4 years ago

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

Updated by Susanne Moog over 4 years ago

  • Related to Feature #89115: Auto-create Redirects on Slug Changes added
Actions #16

Updated by Susanne Moog over 4 years ago

@Sybille: see related issue.

Actions #17

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions #18

Updated by Sybille Peters about 3 years ago

  • Related to Bug #92748: Infinite redirects (redirect loop) when updating slugs added
Actions

Also available in: Atom PDF