From c4eddfdfed40b136298be8bc32d8d31045299930 Mon Sep 17 00:00:00 2001 From: Peter Wilson Date: Tue, 6 Apr 2021 03:22:08 +0000 Subject: [PATCH] Site Health: Reduce false reports of HTTPS failures. Reduce severity of failing HTTPS tests from critical to warning. Stop reporting failures if the site is being accessed over HTTPS but `wp_is_https_supported()` indicates a lack of support. Props annalamprou, AnotherDave, ayeshrajans, bobbingwide, Clorith, dragongate, eatsleepcode, gab81, geoffrey1963, Ipstenu, k3nsai, mmuyskens, nicegamer7, peterwilsoncc, pwallner, SergeyBiryukov, TimothyBlynJacobs, Toru. Fixes #52783. git-svn-id: https://develop.svn.wordpress.org/trunk@50659 602fd350-edb4-49c9-b593-d223f7449a82 --- .../includes/class-wp-site-health.php | 32 +------------------ 1 file changed, 1 insertion(+), 31 deletions(-) diff --git a/src/wp-admin/includes/class-wp-site-health.php b/src/wp-admin/includes/class-wp-site-health.php index 48bd21b6a6..315db5566f 100644 --- a/src/wp-admin/includes/class-wp-site-health.php +++ b/src/wp-admin/includes/class-wp-site-health.php @@ -1528,7 +1528,7 @@ class WP_Site_Health { if ( ! wp_is_using_https() ) { // If the website is not using HTTPS, provide more information about whether it is supported and how it can // be enabled. - $result['status'] = 'critical'; + $result['status'] = 'warning'; $result['label'] = __( 'Your website does not use HTTPS' ); if ( wp_is_site_url_using_https() ) { @@ -1630,36 +1630,6 @@ class WP_Site_Health { ); } } - } elseif ( ! wp_is_https_supported() ) { - // If the website is using HTTPS, but HTTPS is actually not supported, inform the user about the potential - // problems. - $result['status'] = 'critical'; - $result['label'] = __( 'There are problems with the HTTPS connection of your website' ); - - $https_detection_errors = get_option( 'https_detection_errors' ); - if ( ! empty( $https_detection_errors['ssl_verification_failed'] ) ) { - $result['description'] = sprintf( - '

%s

', - sprintf( - /* translators: %s: URL to Settings > General > WordPress Address. */ - __( 'Your WordPress Address is set up to use HTTPS, but the SSL certificate appears to be invalid.' ), - esc_url( admin_url( 'options-general.php' ) . '#siteurl' ) - ) - ); - } else { - $result['description'] = sprintf( - '

%s

', - sprintf( - /* translators: %s: URL to Settings > General > WordPress Address. */ - __( 'Your WordPress Address is set up to use HTTPS, but your website appears to be unavailable when using an HTTPS connection.' ), - esc_url( admin_url( 'options-general.php' ) . '#siteurl' ) - ) - ); - } - $result['description'] .= sprintf( - '

%s

', - __( 'Talk to your web host about resolving this HTTPS issue for your website.' ) - ); } return $result;