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

WordPress Trac noreply at wordpress.org
Thu Jan 21 12:31:59 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 fixed-major  |     Focuses:  rest-api
-----------------------------------+--------------------------------
Changes (by SergeyBiryukov):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"49997" 49997]:
 {{{
 #!CommitTicketReference repository="" revision="49997"
 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.
 }}}

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


More information about the wp-trac mailing list