Changeset 45801

Timestamp:

08/15/2019 12:34:53 AM (6 years ago)

Author:

peterwilsoncc

Message:

Site Health Check: Increase time allowance for cron checks.

Introduces WP_Site_Health::has_late_cron() for late wp-cron jobs and extends the time allowance before a job is considered missed.

In a standard configuration using loopback requests, a job is considered late once past due and missed over five minutes past due.

Late and missed time frames are extended if DISABLE_WP_CRON is defined as true to allow for crontab tasks running less frequently. A job is considered late once it's 15 minutes past due and missed over one hour past due.

A file for site health unit tests has been introduced with tests for cron in critical, late and missed states.

Props rockfire, afragen, peterwilsoncc.
Fixes #47223.

File:

• trunk/src/wp-admin/includes/class-wp-site-health.php (modified) (5 diffs)

trunk/src/wp-admin/includes/class-wp-site-health.php

@@ -12 +12 @@
     private $mysql_rec_version_check;
 
-    public  $is_mariadb                          = false;
+    public $is_mariadb                           = false;
     private $mysql_server_version                = '';
     private $health_check_mysql_required_version = '5.5';
@@ -19 +19 @@
     public $schedules;
     public $crons;
-    public $last_missed_cron = null;
+    public $last_missed_cron     = null;
+    public $last_late_cron       = null;
+    private $timeout_missed_cron = null;
+    private $timeout_late_cron   = null;
 
     /**
@@ -28 +31 @@
     public function __construct() {
         $this->prepare_sql_data();
+
+        $this->timeout_late_cron   = 0;
+        $this->timeout_missed_cron = - 5 * MINUTE_IN_SECONDS;
+
+        if ( defined( 'DISABLE_WP_CRON' ) && DISABLE_WP_CRON ) {
+            $this->timeout_late_cron   = - 15 * MINUTE_IN_SECONDS;
+            $this->timeout_missed_cron = - 1 * HOUR_IN_SECONDS;
+        }
 
         add_filter( 'admin_body_class', array( $this, 'admin_body_class' ) );
@@ -1417 +1428 @@
                 )
             );
-        } else {
-            if ( $this->has_missed_cron() ) {
-                $result['status'] = 'recommended';
-
-                $result['label'] = __( 'A scheduled event has failed' );
-
-                $result['description'] = sprintf(
-                    '<p>%s</p>',
-                    sprintf(
-                        /* translators: %s: The name of the failed cron event. */
-                        __( 'The scheduled event, %s, failed to run. Your site still works, but this may indicate that scheduling posts or automated updates may not work as intended.' ),
-                        $this->last_missed_cron
-                    )
-                );
-            }
+        } elseif ( $this->has_missed_cron() ) {
+            $result['status'] = 'recommended';
+
+            $result['label'] = __( 'A scheduled event has failed' );
+
+            $result['description'] = sprintf(
+                '<p>%s</p>',
+                sprintf(
+                    /* translators: %s: The name of the failed cron event. */
+                    __( 'The scheduled event, %s, failed to run. Your site still works, but this may indicate that scheduling posts or automated updates may not work as intended.' ),
+                    $this->last_missed_cron
+                )
+            );
+        } elseif ( $this->has_late_cron() ) {
+            $result['status'] = 'recommended';
+
+            $result['label'] = __( 'A scheduled event is late' );
+
+            $result['description'] = sprintf(
+                '<p>%s</p>',
+                sprintf(
+                    /* translators: %s: The name of the late cron event. */
+                    __( 'The scheduled event, %s, is late to run. Your site still works, but this may indicate that scheduling posts or automated updates may not work as intended.' ),
+                    $this->last_late_cron
+                )
+            );
         }
 
@@ -1927 +1949 @@
 
         foreach ( $this->crons as $id => $cron ) {
-            if ( ( $cron->time - time() ) < 0 ) {
+            if ( ( $cron->time - time() ) < $this->timeout_missed_cron ) {
                 $this->last_missed_cron = $cron->hook;
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    /**
+     * Check if any scheduled tasks are late.
+     *
+     * Returns a boolean value of `true` if a scheduled task is late and ends processing. If the list of
+     * crons is an instance of WP_Error, return the instance instead of a boolean value.
+     *
+     * @return bool|WP_Error true if a cron is late, false if it wasn't. WP_Error if the cron is set to that.
+     */
+    public function has_late_cron() {
+        if ( is_wp_error( $this->crons ) ) {
+            return $this->crons;
+        }
+
+        foreach ( $this->crons as $id => $cron ) {
+            $cron_offset = $cron->time - time();
+            if (
+                    $cron_offset >= $this->timeout_missed_cron &&
+                    $cron_offset < $this->timeout_late_cron
+                ) {
+                $this->last_late_cron = $cron->hook;
                 return true;
             }