Skip to content

Add Site Health check for Cache-Control: no-store header#2091

Closed
krishnachandak-2004 wants to merge 1 commit into
WordPress:trunkfrom
krishnachandak-2004:feat/site-health-no-store
Closed

Add Site Health check for Cache-Control: no-store header#2091
krishnachandak-2004 wants to merge 1 commit into
WordPress:trunkfrom
krishnachandak-2004:feat/site-health-no-store

Conversation

@krishnachandak-2004
Copy link
Copy Markdown

Fixes #1695

This PR adds a Site Health test that performs a frontend request and checks the Cache-Control header. If it contains no-store, the test recommends avoiding this setting to improve performance for unauthenticated users.

@github-actions
Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Unlinked Accounts

The following contributors have not linked their GitHub and WordPress.org accounts: @krishnachandak-2004.

Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Unlinked contributors: krishnachandak-2004.

Co-authored-by: felixarntz <flixos90@git.wordpress.org>
Co-authored-by: paaljoachim <paaljoachim@git.wordpress.org>
Co-authored-by: westonruter <westonruter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@westonruter
Copy link
Copy Markdown
Member

Hummm. We already have a Site Health test for this: #1807

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Features should be able to be deactivated from the Performance screen

2 participants