Bug #89090: No redirect loop prevention for slugs / redirects - TYPO3 Core - TYPO3 Forge

Custom queries

Accessibility
Easy tasks
Forge Triage
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
My open issues
No feedback within 90 days
Open Bugs
Open issues of 10
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (1)

Wittkiel Gruppe

Actions

Copy link

Bug #89090

closed

No redirect loop prevention for slugs / redirects

Added by Sybille Peters about 5 years ago. Updated almost 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Site Handling, Site Sets & Routing

Target version:

Start date:

2019-09-05

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

System¶

TYPO3 version 9.5.9

Possible Scenario 1 (realistic)¶

Slug for page 15 is /something/other
A redirect gets created to this page: /other -> t3://page?uid=15
Now, the slug is changed fpr page 15 to /other
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:

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

Related issues 2 (1 open — 1 closed)

Related to TYPO3 Core - Feature #89115: Auto-create Redirects on Slug Changes

Closed

Frank Nägler

2019-09-09

Actions

Related to TYPO3 Core - Bug #92748: Infinite redirects (redirect loop) when updating slugs

New

2020-11-02

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Georg Ringer about 5 years ago

There are no redirects created after slug change...

Actions

Copy link

Updated by Sybille Peters about 5 years ago

The scenarios are about creating redirects manually.

Actions

Copy link

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

Copy link

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

Copy link

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

Copy link

Updated by Frank Nägler about 5 years ago

Georg Ringer wrote:

There are no redirects created after slug change...

not yet... wait some days ;)

Actions

Copy link

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

Copy link

Updated by Sybille Peters about 5 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

Copy link

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

Copy link

#10

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

Copy link

#11

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

Copy link

#12

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

Copy link

#13

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

Copy link

#14