Bug #89090
closedNo redirect loop prevention for slugs / redirects
100%
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
Updated by Georg Ringer about 5 years ago
There are no redirects created after slug change...
Updated by Sybille Peters about 5 years ago
The scenarios are about creating redirects manually.
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
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
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
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 ;)
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
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)
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
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
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
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
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
Updated by Andreas Fernandez about 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7de102ce49da90047c2d54ff2545aa000f4ec508.
Updated by Susanne Moog about 5 years ago
- Related to Feature #89115: Auto-create Redirects on Slug Changes added
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed
Updated by Sybille Peters almost 4 years ago
- Related to Bug #92748: Infinite redirects (redirect loop) when updating slugs added