Feature PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
18 Nov 2024

Summary of Changes

The routing in Joomla has had 2 issues in the past: When IDs were enabled in URLs you could basically modify the alias part of the URL at will and if you wanted to switch from ID based URLs to those without, there was no way to automatically redirect everything. This PR tries to fix both issues.

The PR is depending on #44455 to mark a parsed URL as tainted. There are two modes here:

This PR was made possible by the support of djumla. Thank you for that.

IDs switched on

When reading the content item or category segment of a URL, the ID is read and based on that ID the alias is read from the database and if the two don't match, the URL is marked as tainted and later redirected to the "correct" URL. (#43992 is fixing the alias for that URL then) This prevents modified aliases in the URL.

IDs switched off

If IDs are switched off, we are first searching for the normal content item or category as we are doing now, but when we don't find a match, we assume that we are actually reading a legacy URL with IDs switched on. In that case we mark the URL as tainted (regardless of it being totally broken or not, since when we fail to parse the URL at all, we already throw a 404 before we reach the handling of tainted URLs) and then compare the segment with the segment we would get with IDs enabled. If that matches, we return the ID and later redirect to the correct URL. This means that you can switch from ID-based URLs to "clean" URLs and Joomla will automatically redirect all pages to the correct URLs.

Testing Instructions

  1. Apply both this PR and #44455 and switch on IDs in the integration tab of either com_content, com_contact or com_newsfeeds. Surf to URLs in your frontend containing IDs and modify the alias of either the category or the content item (article, contact or newsfeed) and load that URL.
  2. Disable IDs for the component and reload the ID-based URL

Actual result BEFORE applying this Pull Request

for 1.: You get the same content with more than one URL
for 2.: You get a 404 error

Expected result AFTER applying this Pull Request

for 1.: You are redirected to the right URL again with the correct alias.
for 2.: You are redirected to the right URL without IDs

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar Hackwar Hackwar - open - 18 Nov 2024
avatar Hackwar Hackwar - change - 18 Nov 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Nov 2024
Category Front End com_contact com_content com_newsfeeds
avatar brianteeman brianteeman - test_item - 19 Nov 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 19 Nov 2024

I have tested this item ✅ successfully on 1e5d91a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44477.

avatar SniperSister SniperSister - test_item - 21 Nov 2024 - Tested successfully
avatar SniperSister
SniperSister - comment - 21 Nov 2024

I have tested this item ✅ successfully on 1e5d91a

Works as described


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44477.

avatar Hackwar Hackwar - change - 21 Nov 2024
Labels Added: Feature PR-5.3-dev
avatar Hackwar Hackwar - alter_testresult - 21 Nov 2024 - SniperSister: Tested successfully
avatar Hackwar Hackwar - alter_testresult - 21 Nov 2024 - brianteeman: Tested successfully
avatar Hackwar Hackwar - change - 21 Nov 2024
Status Pending Ready to Commit
avatar Hackwar
Hackwar - comment - 21 Nov 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44477.

Add a Comment

Login with GitHub to post a comment