Changeset 49997

Timestamp:

01/21/2021 12:31:44 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Site Health: Use a front-end URL for loopback tests.

In [49154] the async Site Health tests were changed to use the REST API instead of admin-ajax. An unintended side effect of this change was that the loopback tests which tried to ping the site's admin_url() were no longer authenticated because admin-cookies aren't provided to the REST API.

This commit adjusts the loopback test to use the front-end site_url which checks that cron will function properly. A follow-up ticket will focus on tests that will cover the file editor checks.

Props Clorith.
Merges [49917] to the 5.6 branch.
Fixes #52097.
See #48105.

Location:

branches/5.6

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/class-wp-site-health.php (modified) (1 diff)

branches/5.6/src/wp-admin/includes/class-wp-site-health.php

@@ -2532 +2532 @@
         }
 
-        $url = admin_url();
+        $url = site_url();
 
         $r = wp_remote_get( $url, compact( 'cookies', 'headers', 'timeout', 'sslverify' ) );