Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function mcrypt_get_iv_size() is deprecated #466

Closed
SLiCK-303 opened this issue Feb 19, 2018 · 6 comments
Closed

Function mcrypt_get_iv_size() is deprecated #466

SLiCK-303 opened this issue Feb 19, 2018 · 6 comments
Labels
Code Quality Outdated Issue is no longer relevant due to significant changes in the codebase

Comments

@SLiCK-303
Copy link
Contributor

SLiCK-303 commented Feb 19, 2018

I get the following error on the site, with php 7.1.x...
mod_fcgid: stderr: PHP Deprecated: Function mcrypt_get_iv_size() is deprecated in /classes/Rijndael.php on line 75

@firstred
Copy link
Contributor

Thanks for the issue. It should be addressed with 1.0.4. Have you tested it on 1.0.4?

@SLiCK-303
Copy link
Contributor Author

SLiCK-303 commented Feb 22, 2018

Yes, this is latest source I was talking about. Sorry I didnt say what tb version I was using.

On 1.0.3 I get..
mod_fcgid: stderr: PHP Deprecated: Function mcrypt_decrypt() is deprecated in /classes/Rijndael.php on line 102

Both with debugging on, if it matters

@Traumflug
Copy link
Contributor

@Captain-FLAM
Copy link
Contributor

So I decided to push it there :

4e76040

@Traumflug
Copy link
Contributor

This method is still used in some code, in classes/Rijndael.php and install-dev/models/install.php. Plan is to remove all encryption variants but the one provided by PHP's OpenSSL, for less code and removal of a switch a merchant has to decide about.

@getdatakick
Copy link
Contributor

Outdated - Rijndael was removed from core

@getdatakick getdatakick added the Outdated Issue is no longer relevant due to significant changes in the codebase label Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Quality Outdated Issue is no longer relevant due to significant changes in the codebase
Projects
None yet
Development

No branches or pull requests

5 participants