[#45646] - Serve correct 404 status for URLs with invalid controller name
- Fixed in Code Base
- 1 Aug 2025
- Medium
- Build: 5.3-dev
- # 45646
- Diff
- Denitz:patch-21
User tests: Successful: Unsuccessful:
Summary of Changes
Currently, if a page is loaded with task handling invalid controller, Joomla reports 500 server error status.
But logically, it should be a 404 response, but not a server error.
For instance, I have a custom notifier on server errors, but it's just an invalid URL, or it could be an old URL with menu item type changes. Once indexed by search engine, it will send you 500 errors.
Testing Instructions
Load any usual single article URL with appended invalid task like ?task=foo.bar, i.e.:
https://www.joomla.org/announcements/release-news/5928-joomla-5-3-1-bugfix-release.html?task=foo.bar
Actual result BEFORE applying this Pull Request
Invalid URL returns 500 status.
Expected result AFTER applying this Pull Request
Invalid URL returns 404 status.
Link to documentations
Please select:
-
No documentation changes for docs.joomla.org needed
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
I have tested this item ✅ successfully on 963fbd1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45646.
Before
After
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
PR-5.3-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45646.
I have tested this item ✅ successfully on 963fbd1
I have tested this successfully... thanks @Denitz!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45646.
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-08-01 15:12:01 |
| Closed_By | ⇒ | rdeutz |
I have tested this item ✅ successfully on 963fbd1
Tested with JBT, before applying this PR:
500 Internal Server Errorhttp://localhost:7053/index.php/component/content/article/first?task=foo.barAfter applying the PR:
404 Not Found404 Not Foundfor combined parameter first?catid=2&task=foo.barThis comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45646.