Skip to content

fix: use Vercel-supported cache directive#33

Merged
corwinm merged 1 commit into
mainfrom
fix/cache-control-s-maxage
Jul 1, 2026
Merged

fix: use Vercel-supported cache directive#33
corwinm merged 1 commit into
mainfrom
fix/cache-control-s-maxage

Conversation

@corwinm

@corwinm corwinm commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix route Cache-Control headers to use Vercel-supported s-maxage instead of s-max-age
  • Update page, blog post, RSS, sitemap, and robots headers consistently
  • Add a Vitest regression test that prevents the unsupported directive from returning

Test Plan

  • pnpm vitest run app/lib/cache-control.test.ts failed before the header fix and passed after it
  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm build
  • Production server smoke test with PORT=4173 pnpm exec react-router-serve ./build/server/nodejs_eyJydW50aW1lIjoibm9kZWpzIn0/index.js; verified cache-control contains s-maxage on /, /resume, /wishlist, /rss.xml, /sitemap.xml, /robots.txt, and /blog/arashi

@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
corwinm-remix Ready Ready Preview, Comment Jul 1, 2026 7:32pm

@corwinm
corwinm merged commit 433c52d into main Jul 1, 2026
3 checks passed
@corwinm
corwinm deleted the fix/cache-control-s-maxage branch July 1, 2026 20:19
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