[Cache] ELI5: Simplify troubleshooting docs#30638
Open
Oxyjun wants to merge 2 commits intoproductionfrom
Open
Conversation
Apply ELI5 analysis to improve readability across 4 pages: - index.mdx: Minor intro wording improvement - always-online.mdx: Add intro context, fix broken list item, clarify 520-527 vs origin errors distinction, convert FAQ questions to headings, fix style (we -> Cloudflare, Cache-Control formatting) - dynamic-content-and-login-issues.mdx: Fix Set-Cookie removal explanation to match source docs, expand CSRF acronym, move cross-reference inline - mp4-videos-on-ios-and-safari.mdx: Add Why this happens section explaining range requests and ETags, move rule ordering explanation before steps
Contributor
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
Contributor
Oxyjun
commented
May 7, 2026
Address PR review comment: FAQ section headings should be questions since they appear under Frequently asked questions. Restored question format while keeping the improved answer content.
ranbel
reviewed
May 7, 2026
| ## Best practices | ||
|
|
||
| - **Allow requests from the Internet Archive IP addresses.** The Internet Archive crawler sends requests directly to your origin server. Make sure you are not blocking the Internet Archive IP ranges: `207.241.224.0/20` and `208.70.24.0/21`. | ||
| - **The Internet Archive does not consider your origin server's `Cache-Control` header.** The Internet Archive archives pages regardless of `Cache-Control` directives because it archives content rather than caching it. |
Contributor
There was a problem hiding this comment.
consider reframing this bullet in terms of what the user should do or not do
ranbel
approved these changes
May 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docs/cache/troubleshooting/Changes by file
index.mdxalways-online.mdxCache-Controlformatting)dynamic-content-and-login-issues.mdxSet-Cookieremoval explanation to match source docs (removal happens because Edge TTL forces caching, not as a separate prior step), expand CSRF acronym on first use, move cross-reference link inlinemp4-videos-on-ios-and-safari.mdxReview notes
All changes validated through adversarial review. No unsourced claims. Key fix:
always-online.mdxhad a broken list item and conflated origin-returned errors with origin-unreachable errors.