• Resolved blackhawkcybersec

    (@blackhawkcybersec)


    Name: Query Monitor

    FILE: /XXXXXXXX/wp-content/plugins/query-monitor/collectors/environment.php
    ————————————————————————————————————–
    FOUND 1 ERROR AFFECTING 1 LINE
    ————————————————————————————————————–
    131 | ERROR | Extension ‘mysql_’ is deprecated since PHP 5.5 and removed since PHP 7.0 – use mysqli instead.
    ————————————————————————————————————–

    Query Monitor IS PHP 7 compatible. Your plugin inaccurately marks another top plugin as not compatible.

    Please fix.

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Contributor Jason Stallings

    (@octalmage)

    Unfortunately static code analysis will not be able to detect this one. I’ll add Query Monitor to the Whitelist.

    Thanks!

    Thread Starter blackhawkcybersec

    (@blackhawkcybersec)

    Unfortunately static code analysis will not be able to detect this one.

    While I do agree with you in general about static code analysis, in cases like this you may be able to add some tweaks to the algorithm to avoid this.

    Some of the false positives we’ve seen relate to detection of functions, capabilities, settings, and they are getting flagged.

    You don’t have to stick with a stock version of the library you’re using…you can tweak the algorithm to make it smarter.

    That being said, thank you for whitelisting it.

    Thread Starter blackhawkcybersec

    (@blackhawkcybersec)

    @octalmage,

    You mentioned that you had whitelisted this plugin, but the false positive still exists: Query Monitor is still coming up as not PHP 7 compatible in your latest version, 1.3.0.

    Plugin Contributor Jason Stallings

    (@octalmage)

    Query Monitor is on the whitelist with v1.3.1, thanks again for the report!

    Thread Starter blackhawkcybersec

    (@blackhawkcybersec)

    Thank you. Confirmed..it is being detected as PHP 7 compatible now.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘FALSE POSITIVE: Showing Query Monitor as not PHP 7 Compatible’ is closed to new replies.