Integrate with Site Health Check

[westonruter]

There are some opportunities for the AMP plugin to integrate with the Site Health Check in 5.2, including both status and info:

• Detect whether loopback requests are working. (Done ✅)
• Whether persistent object cache is present.
• Warn when suggested extensions are not installed (Remove intl from required extensions; add add date/spl as required; add others to suggest #2190).
• Warn when curl_multi_* functions aren't available (Remove requirement that curl multi functions exist #2423).
• Flag themes and plugins that are not AMP-compatible. (Future)
• What mode is enabled (native/transitional/reader).
• What experiences are enabled.
• Which templates are enabled.
• Misc AMP plugin settings.
  ...

We should include as much information as is helpful to debug issues that are reported on the forums.

[swissspidy]

Detect whether loopback requests are working.

Site Health does that by default because of cron. Perhaps that check is already enough.

[westonruter]

See also #2006 (Add warning when a site is not served via HTTPS)

[westonruter]

Every warning that we're showing in the plugin admin screen should also be shown in Site Health.

[kienstra]

Request To Work On This

Hi @westonruter,
Hope your day's off to a great start.

Could I work on this? Thanks!

[westonruter]

Yes, please.

[westonruter]

Something that has come up twice recently (1, 2, 3) is a deprecation notice:

PHP Deprecated: idn_to_utf8(): INTL_IDNA_VARIANT_2003 is deprecated in ...amp/includes/class-amp-http.php on line 233

We may want to consider adding a Site Health check for outdated ICU version. /cc @schlessera

It was noted by @swissspidy that:

This typically happens when the version of PHP has been compiled with an older version of ICU.

[westonruter]

Something else which may end up needing to be tested for is whether zlib output compression is enabled, as apparently this breaks the output buffer processing. Awaiting to confirm. See https://wordpress.org/support/topic/links-and-buttons-are-disabled/#post-12161184

[kienstra]

Testing Steps

1. Go to, /wp-admin/site-health.php, and ensure the 'Status' tab is showing
2. Expected: there should be 3 AMP pillboxes:

In the 'Passed tests' section:

3. Click the 'Info' tab at the top, or go to /wp-admin/site-health.php?tab=debug
4. Expected: Something like this should display:

5. Make some changes to the 'AMP Settings' page. For example, disable Pages:

6. Go back to the Site Health 'Info' tab in Step 3
7. Expected: the change is reflected there. For example:

8. Repeat steps 5-7 for several different settings

[csossi]

Verified in QA