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

Jump to bottom

PHP Unknown: Function mcrypt_decrypt() is deprecated #5126

Closed
eka7a opened this issue Jan 6, 2017 · 7 comments
Closed

PHP Unknown: Function mcrypt_decrypt() is deprecated #5126

eka7a opened this issue Jan 6, 2017 · 7 comments
Assignees
@danielkerr
Labels
In Progress We are currently working on a fix/update

Comments

@eka7a
Copy link
Collaborator

eka7a commented Jan 6, 2017

Mcrypt extension has been deprecated in PHP 7.1
http://php.net/manual/en/migration71.deprecated.php

Instead of Mcrypt, can be used openssl or a different method.
http://php.net/manual/en/function.openssl-encrypt.php
@arnisjuraga
Copy link
Contributor

Hmm.... openssl-encrypt is available from 5.3 (which is minimal OC requirements too), so - for compatibility, you are right - could be replaced with openssl. Can you provide a pull request?

@danielkerr
Copy link
Member

please

@eka7a
Copy link
Collaborator Author

eka7a commented Jan 8, 2017

@arnisjuraga
At least PHP 5.4 is now required for OpenCart.
https://github.com/opencart/opencart/blob/master/upload/install/controller/install/step_2.php#L107

@danielkerr ok. I will send a pull request.

@arnisjuraga
Copy link
Contributor

Sorry, you are right. I was referencing to the documentation. So, @danielkerr please update this #5132

eka7a added a commit to eka7a/opencart that referenced this issue Jan 10, 2017
@eka7a
Update Encryption Method
040d6ba 
opencart#5126
eka7a added a commit to eka7a/opencart that referenced this issue Jan 10, 2017
@eka7a
Update Encryption Method
9ecceed 
opencart#5126
@eka7a eka7a mentioned this issue Jan 10, 2017
Closed
@ADDCreative
Copy link
Contributor

Don't forget to update the installers and OpenBay.
https://github.com/opencart/opencart/search?utf8=%E2%9C%93&q=mcrypt

@arnisjuraga
Copy link
Contributor

#5132

@juanmalopez juanmalopez mentioned this issue Jan 26, 2017
Closed
@danielkerr danielkerr self-assigned this May 8, 2017
@danielkerr danielkerr added the In Progress We are currently working on a fix/update label May 8, 2017
danielkerr added a commit that referenced this issue May 8, 2017
@danielkerr
https://github.com/opencart/opencart/issues/5126
2773a22
@danielkerr
Copy link
Member

already fixed yesterday to use openssl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danielkerr danielkerr

Labels
In Progress We are currently working on a fix/update
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ADDCreative @arnisjuraga @danielkerr @eka7a