Skip to content

Fix stale HTML + prevent indexing of backups#26

Open
SandLotLab wants to merge 1 commit into
mainfrom
seo-cache-cleanup
Open

Fix stale HTML + prevent indexing of backups#26
SandLotLab wants to merge 1 commit into
mainfrom
seo-cache-cleanup

Conversation

@SandLotLab

Copy link
Copy Markdown
Owner
  • Force fresh HTML (no stale cache from Cloudflare)
  • Keep assets cached for performance
  • Prevent /backups/ from being indexed by Google
  • Ensure sitemap + robots are always fresh

This directly addresses stale pages showing in Google and improves crawl accuracy.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2f53392159

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread _headers
Comment on lines +2 to +5
Cache-Control: public, max-age=0, must-revalidate

/assets/*
Cache-Control: public, max-age=31536000, immutable

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Avoid combining contradictory Cache-Control rules

This global Cache-Control rule also matches /assets/*, and on Cloudflare Pages multiple matching _headers rules are merged with duplicate header values comma-joined, so assets will receive both max-age=0, must-revalidate and max-age=31536000, immutable in the same header value. That makes asset caching behavior ambiguous and can defeat the intended long-lived asset cache policy.

Useful? React with 👍 / 👎.

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.

1 participant