[wp-trac] [WordPress Trac] #52097: Site Health Loopback Test doesn't send admin cookies

WordPress Trac noreply at wordpress.org
Sat Jan 2 20:07:41 UTC 2021


#52097: Site Health Loopback Test doesn't send admin cookies
-------------------------------+--------------------------------
 Reporter:  TimothyBlynJacobs  |       Owner:  TimothyBlynJacobs
     Type:  defect (bug)       |      Status:  closed
 Priority:  normal             |   Milestone:  5.6.1
Component:  Site Health        |     Version:  5.6
 Severity:  normal             |  Resolution:  fixed
 Keywords:  has-patch          |     Focuses:  rest-api
-------------------------------+--------------------------------
Changes (by TimothyBlynJacobs):

 * owner:  (none) => TimothyBlynJacobs
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"49917" 49917]:
 {{{
 #!CommitTicketReference repository="" revision="49917"
 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.
 Fixes #52097.
 See #48105.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/52097#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list