? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
23 Mar 2019

Summary of Changes

This is an further protection (requires PHP 5.6+) about the session timing attack implemented in Joomla 3.8.8

This issue is now moved to the public tracker thanks for the work done on this by:
@demis-palma
@wilsonge
@SniperSister
@mbabker
@PhilETaylor

Testing Instructions

  • apply the patch
  • login the backend
  • make sure the ajax call checking for Joomla Updates still works as expected.

Expected result

Everything works and session is aborted

Actual result

In some very very rare cases session is not aborted.

Documentation Changes Required

none.

avatar zero-24 zero-24 - change - 23 Mar 2019
Milestone Added:
avatar zero-24 zero-24 - open - 23 Mar 2019
avatar zero-24 zero-24 - change - 23 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Mar 2019
Category Administration com_installer
avatar zero-24 zero-24 - change - 23 Mar 2019
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 23 Mar 2019

Get session from application please. Joomla\CMS\Factory::getSession() is deprecated.

avatar zero-24
zero-24 - comment - 24 Mar 2019

Ok ready for testing now. George just corrected the call and I removed the now useless use call :)

avatar SharkyKZ
SharkyKZ - comment - 31 Mar 2019

I have tested this item successfully on a0fed8a


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

avatar SharkyKZ SharkyKZ - test_item - 31 Mar 2019 - Tested successfully
avatar zero-24 zero-24 - change - 17 Apr 2019
Labels Removed: J4 Issue
avatar zero-24
zero-24 - comment - 18 May 2019

Can we get one more test here please? cc @wilsonge @SniperSister

avatar zero-24
zero-24 - comment - 16 Jun 2019

Would be great when we can get one more tester so we can include this patch in 4.0. :)

avatar zero-24
zero-24 - comment - 20 Jul 2019

Can we find anyhow an tester on this one?

avatar wilsonge wilsonge - change - 13 Sep 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-09-13 12:13:25
Closed_By wilsonge
avatar wilsonge wilsonge - close - 13 Sep 2019
avatar wilsonge wilsonge - merge - 13 Sep 2019
avatar wilsonge
wilsonge - comment - 13 Sep 2019

I'm merging this so that it gets extra testing.

Add a Comment

Login with GitHub to post a comment