[#42473] - Route the redirect in the menu logout
- Fixed in Code Base
- 17 Jan 2024
- Medium
- Build: 4.4-dev
- # 42473
- Diff
- Digital-Peak:menu/logout
User tests: Successful: Unsuccessful:
Summary of Changes
The logout via a menu item should route the redirect to logout task. This error happens on one of my production site but not on a fresh Joomla. So it is probably a configuration issue. Neverthless, all controller actions in this class are routing the requests, so should the menu logout as well.
Testing Instructions
- Create a logout menu item
- Log in on the front end
- Click on the logout menu item
- Additionally this can also be tested with the htaccess redirect
Actual result BEFORE applying this Pull Request
Logout doesn't work on some sites.
Expected result AFTER applying this Pull Request
Logout through a menu item works direct.
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_users |
| Labels |
Added:
PR-4.4-dev
|
||
Thanks @brianteeman for the reference. Can you give it a test? So we can ship with the next release.
I have tested this item ? unsuccessfully on 501ae72
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42473.
Sorry for the delay in responding - I had to set up a j4 dev again
Using the detailed test instructions that I wrote in #37763
Before this PR I can no longer replicate the issue in that pr so it must have been solved elsewhere and I have closed it.
With this PR both tests for logging out with the menu item FAIL
Warning
The security token did not match. The request was cancelled to prevent any security breach. Please try again.
Can you test again by clearing the cookies first. This error comes mostly from the situation where the session has expired before doing the action.
That was done at the time. Here is a recording
@brianteeman found the issue, can you test again?
Sorry for the delay in responding - I had to set up a j4 dev again
Using the detailed test instructions that I wrote in #37763
Before this PR I can no longer replicate the issue in that pr so it must have been solved elsewhere and I have closed it.
With this PR both tests for logging out with the menu item FAIL
Warning
The security token did not match. The request was cancelled to prevent any security breach. Please try again.
Repeated the tests.
Still can't replicate the original error
Still can confirm the new error with the earlier version of this pr
Applied the current version of this pr and got no errors
Thanks for the test, I could also only reproduce the issue on one of my sites I'm maintaining.
Thanks for the test, I could also only reproduce the issue on one of my sites I'm maintaining.
Does the error also occur when you add a new logout menu item to that site?
It happens only then.
No message, it just doesn't log out.
No message, it just doesn't log out.
I remember having the same issue with a site in the past. I can no longer reproduce it now.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-01-17 11:47:56 |
| Closed_By | ⇒ | MacJoom |
See #37763