WordPress File Monitor Plus
[resolved] Small bug in Other Cron code (3 posts)

  1. ionainteractive
    Posted 3 years ago #

    There is a small bug in line #137 of classes/wpfmp.class.php (function external_cron_run_check). When checking the $_GET parameters to decide whether or not to process an external cron request, you currently have "isset($_GET['sc_wpfmp_key'])" as a disqualifying expression, but it's negation should be the disqualifier, because that parameter MUST be presented (and match the back-end security_key option value) in order to process. So that expression simply needs an exclamation point before it, to be:

    if( ! isset( $_GET['sc_wpfmp_scan'] ) || ! isset( $_GET['sc_wpfmp_key'] ) || 1 != $_GET['sc_wpfmp_scan'] || $options['security_key'] != $_GET['sc_wpfmp_key'] || "other" != $options['cron_method'] )


  2. Scott Cariss
    Plugin Author

    Posted 3 years ago #

    Thankyou very much for reporting this. I made this mistake when refactoring the code *doh*. I have pushed an update to the repository 2.0.1 hopefully this fixes it.

    I hope this bug wasn't to hard to track down and I'm sorry if it caused you any inconvenience.

    Kind Regards


  3. ionainteractive
    Posted 3 years ago #

    No worries! Glad to help resolve! :-)

Topic Closed

This topic has been closed to new replies.

About this Plugin

  • WordPress File Monitor Plus
  • Frequently Asked Questions
  • Support Threads
  • Reviews

About this Topic