?
avatar PhilETaylor
PhilETaylor
10 May 2021

Steps to reproduce the issue

To EASILY replicate, edit libraries/src/Session/SessionManager.php and put

throw new \Exception();

on line 65 in the destroySessions method, and then LOGOUT of Joomla admin to cause the crash

Expected result

An error displayed to the screen and the admin template to survive intact with the left bar correct.

Actual result

Screenshot 2021-05-20 at 22 03 18

avatar PhilETaylor PhilETaylor - open - 10 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 10 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 10 May 2021
avatar PhilETaylor PhilETaylor - change - 10 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 10 May 2021
avatar richard67
richard67 - comment - 10 May 2021

I think it had the same cause as #33740 . will check later today or tomorrow morning.

avatar PhilETaylor
PhilETaylor - comment - 10 May 2021

You are confused :) haha - this issue was forked from #33740 so that the display of the admin could be fixed independently of the pgsql issue with sessions haha

avatar richard67
richard67 - comment - 10 May 2021

Yes, I should go sleep soon.

avatar brianteeman
brianteeman - comment - 20 May 2021

As #33788 was resolved can I assume this one is as well or do I need to install postgres to check?

avatar PhilETaylor
PhilETaylor - comment - 20 May 2021

No this one is still a problem.

To EASILY replicate, edit libraries/src/Session/SessionManager.php and put

throw new \Exception();

on line 65 in the destroySessions method, and then LOGOUT of Joomla admin to cause the crash

Screenshot 2021-05-20 at 22 03 18

avatar PhilETaylor PhilETaylor - change - 20 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 20 May 2021
avatar PhilETaylor
PhilETaylor - comment - 20 May 2021

I have updated the issue description

avatar brianteeman
brianteeman - comment - 20 May 2021

thanks - replicated it and now looking at a fix

avatar brianteeman
brianteeman - comment - 20 May 2021

please test #34061

avatar richard67 richard67 - change - 20 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-20 21:28:32
Closed_By richard67
avatar richard67 richard67 - close - 20 May 2021
avatar richard67
richard67 - comment - 20 May 2021

Closing as having a pull request. Please test #34061 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment