[wp-trac] [WordPress Trac] #52783: Health Check mis-reports https functionality in certain situations

[WordPress Trac]

#52783: Health Check mis-reports https functionality in certain situations
--------------------------------------+----------------------------
 Reporter:  Ipstenu                   |       Owner:  peterwilsoncc
     Type:  defect (bug)              |      Status:  assigned
 Priority:  normal                    |   Milestone:  5.7.1
Component:  Site Health               |     Version:  5.7
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+----------------------------
Changes (by peterwilsoncc):

 * owner:  (none) => peterwilsoncc
 * status:  new => assigned


Comment:

 Replying to [comment:33 SergeyBiryukov]:
 > Replying to [comment:29 peterwilsoncc]:
 > > (As a maybe) Send basic auth headers in
 `wp_update_https_detection_errors()` if the site is currently been
 accessed behind basic auth.
 >
 > I'm not too sure about this last part at a glance, would need to take a
 deeper look. The rest of the PR looks good to me :)

 Thanks Sergey, I've reverted the basic auth portion of the linked pull
 request in order to get this in without been side-tracked. I've opened
 #52977 as a follow up ticket for whether basic auth headers ought to be
 included.

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