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

Error S2member Framework #243

Open
Medicareschool opened this issue Jul 1, 2019 · 5 comments
Open

Error S2member Framework #243

Medicareschool opened this issue Jul 1, 2019 · 5 comments

Comments

@Medicareschool
Copy link

Currently running S2member v. 190617. It seems to have no issues, but PHP 7.2 returned errors. False?
Report:
FILE: XXXXXXXXX/public_html/wp-content/plugins/s2member/src/vendor/paragonie/random_compat/lib/random_bytes_mcrypt.php

FOUND 3 ERRORS AFFECTING 1 LINE

58 | ERROR | Function mcrypt_create_iv() is deprecated since PHP 7.1 and removed since PHP 7.2; Use random_bytes() or OpenSSL instead
58 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
58 | ERROR | The constant "MCRYPT_DEV_URANDOM" is deprecated since PHP 7.1 and removed since PHP 7.2

XXXXXXXXXXX/html/wp-content/plugins/s2member/src/vendor/paragonie/random_compat/lib/byte_safe_strings.php

FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES

32 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP 7.2
86 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP 7.2

@jrfnl
Copy link

jrfnl commented Jul 1, 2019

This has already been fixed in the latest version of PHPCompatibilityWP. This plugin just needs to upgrade to use a more recent version of the underlying technology.

@Medicareschool
Copy link
Author

Medicareschool commented Jul 1, 2019 via email

@jrfnl
Copy link

jrfnl commented Jul 1, 2019

@Medicareschool If those are the only issues reported, you should be fine.

@jrfnl
Copy link

jrfnl commented Jul 1, 2019

@Medicareschool I was posting that a bit too fast - sorry.

As the underlying tech is so out of date, a lot of PHP 7.3 issues aren't scanned for with this plugin (yet), so there's no telling based on the above results.

If you want to get a more accurate scan, you can use the underlying technology directly to scan your installation with the up-to-date version: https://github.com/PHPCompatibility/PHPCompatibilityWP

@tacodicaprio
Copy link

Latest PHP Compatibility Checker also reported the mcrypt errors for PHP 7.3, but according to the plugin author those are only loaded for older PHP versions. So, false positive it is.

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

No branches or pull requests

3 participants