User tests: Successful: Unsuccessful:
The mcrypt extension has been abandonware for over a decade now, and was also fairly complex to use. It has therefore been deprecated in favour of OpenSSL, where it will be removed from the core and into PECL in PHP 7.2. It was deprecated in PHP 7.1
There is one use in Joomla 4 that can still be triggered, and that is to decrypt the configuration and OTEP values from the database, if previously encrypted with mcrypt.
If this case is triggered then Joomla decrypts using mcrypt and re-encrypts using openssl using the AES Class.
THIS CODE HAS NEVER WORKED ANYWAY SINCE BEING ADDED due to a typo in the constructor arguments when setting up the $mcrypt
instance. The number of params provided meant that the mcrypt' value last in the params list was never used, meaning that
$mcrypt` was always an instance using the openssl adapter.
As Joomla 4.0.0 is about to be released (cough, maybe some time, but the RC will be) we need to slip these deprecated tags in else it will be another decade before we can remove this code.
@wilsonge urgent review please :) :) :)
Code review.
// cc @nikosdion @joomdonation
// replacement for #33954
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_users Libraries |
Labels |
Added:
?
|
drone failure unrelated to PR content :)
More than happy to merge the deprecation stuff. Honestly the constructor thing is one of those things where - do we break things more by fixing it than it was before xD #scaredMaintainer :)
well its clearly never worked ever... so maybe I just document that it has never ever worked and put verbose notes to say its never ever worked and a note to say to remove it in 5.0.0 rather than fix it.
What it means is that any OTP config, currently encrypted with mcrypt, will not be able to be decrypted and migrated to openssl. Its a very very fringe case anyway.
Obviously the correct course of action is to fix it properly. :-)
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-24 09:48:25 |
Closed_By | ⇒ | PhilETaylor |
I have tested this item✅ successfully on e9ab4ee
Code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34147.