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

Update extension to not use mcrypt directly but rather mcryptcompat (magento core 194x) or openssl #66

Closed
seansan opened this issue Sep 30, 2020 · 1 comment

Comments

@seansan
Copy link

seansan commented Sep 30, 2020

To Reproduce
Steps to reproduce the behavior:

  • php 7.2 Openmage LTS

Expected result

  • Extension does no direct calls to mcrypt

Actual result

  • Mcrypt warnings in log and mcrypt usage => need to remove it from the code asap

Solution

  • Update extension to not use mcrypt directly but rather mcryptcompat (magento core 194x) or openssl

OpenMage/magento-lts#129

https://github.com/OpenMage/magento-lts/blob/1.9.4.x/app/Mage.php#L50

Screenshots

[28-Sep-2020 21:34:40 UTC] PHP Deprecated: Function mdecrypt_generic() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 133
[29-Sep-2020 09:35:24 UTC] PHP Deprecated: Function mdecrypt_generic() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 133
[29-Sep-2020 09:35:24 UTC] PHP Deprecated: Function mdecrypt_generic() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 133
[30-Sep-2020 09:33:04 UTC] PHP Deprecated: Function mcrypt_module_open() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 77
[30-Sep-2020 09:33:04 UTC] PHP Deprecated: Function mcrypt_enc_get_iv_size() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 86
[30-Sep-2020 09:33:04 UTC] PHP Deprecated: Function mcrypt_create_iv() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 86
[30-Sep-2020 09:33:04 UTC] PHP Deprecated: Function mcrypt_enc_get_key_size() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 90
[30-Sep-2020 09:33:04 UTC] PHP Deprecated: Function mcrypt_generic_init() is deprecated in lib/Varien/Crypt/Mcrypt.php on line 97

@seansan seansan changed the title [BUG] Please summarize your bug issue here Update extension to not use mcrypt directly but rather mcryptcompat (magento core 194x) or openssl Sep 30, 2020
@tig-jaspersmits
Copy link
Contributor

Hello @seansan ,

When answering your other case I stumbled upon this one. As mentioned in the other issue as well, we unfortunately can't implement any feature requests like this any more. I'm sorry to say I would need to close this request as our focus is on the M2 extension since M1 has reached EOL.

Kind regards,
Jasper
TIG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants