Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Update cache-control headers in Firebase transformer #83

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

warnyul
Copy link
Owner

@warnyul warnyul commented Dec 9, 2024

  • Adjusted Cache-Control headers in transformer-firebase.js:

    • Added max-age=31536000 to complement s-maxage=31536000 and ensure consistent cache behavior across all levels.
    • Ensures better compatibility and optimized caching for static assets.
  • Removed commented-out unused code for cleaner and more maintainable configuration.

  • Impact:

    • Improves caching efficiency for assets by aligning cache-control directives.
    • Simplifies the Firebase hosting configuration for enhanced clarity.

- Adjusted `Cache-Control` headers in `transformer-firebase.js`:
  - Added `max-age=31536000` to complement `s-maxage=31536000` and ensure consistent cache behavior across all levels.
  - Ensures better compatibility and optimized caching for static assets.

- Removed commented-out unused code for cleaner and more maintainable configuration.

- **Impact**:
  - Improves caching efficiency for assets by aligning cache-control directives.
  - Simplifies the Firebase hosting configuration for enhanced clarity.
@warnyul warnyul self-assigned this Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

Visit the preview URL for this PR (updated for commit f2fa5ac):

https://bvarga-fe600--pr83-cache-fix-pcffcesb.web.app

(expires Mon, 16 Dec 2024 06:53:01 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a3b1098a9eaf0750a134a339972f1df2a3969a8d

@warnyul warnyul merged commit a781062 into main Dec 9, 2024
2 checks passed
@warnyul warnyul deleted the cache-fix branch December 9, 2024 06:54
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