Bug #89090

No redirect loop prevention for slugs / redirects

Added by Sybille Peters over 1 year ago. Updated over 1 year 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

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

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

Actions
#1

Updated by Georg Ringer over 1 year ago

There are no redirects created after slug change...

#2

Updated by Sybille Peters over 1 year ago

The scenarios are about creating redirects manually.

#3

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

#4

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

#5

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

#6

Updated by Frank Naegler over 1 year ago

Georg Ringer wrote:

There are no redirects created after slug change...

not yet... wait some days ;)

#7

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

#8

Updated by Sybille Peters over 1 year 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)

#9

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

#10

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

#11

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

#12

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

#13

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

#14

Updated by Andreas Fernandez over 1 year ago

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

Updated by Susanne Moog over 1 year ago

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

Updated by Susanne Moog over 1 year ago

@Sybille: see related issue.

#17

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed
#18

Updated by Sybille Peters 3 months ago

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

Also available in: Atom PDF