Skip to content

Conversation

@shoyuf
Copy link

@shoyuf shoyuf commented Dec 2, 2025

  • add explicit support for zstd in Accept-Encoding parsing
  • set preferredEncodings order to [zstd, br, gzip, deflate]
  • keep wildcardAcceptEncoding for Accept-Encoding: * limited to gzip and deflate
  • update error message and directive regex to include zstd
  • adjust tests to reflect new zstd preference and wildcard behavior
  • clarify default behavior when no Accept-Encoding header is sent
  • document zstd support and encoding priorities in the README

Checklist

  • I have ensured my pull request is not behind the main or master branch of the original repository.
  • I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
  • I have written a commit message that passes commitlint linting.
  • I have ensured that my code changes pass linting tests.
  • I have ensured that my code changes pass unit tests.
  • I have described my pull request and the reasons for code changes along with context if necessary.

- add explicit support for zstd in Accept-Encoding parsing
- set preferredEncodings order to [zstd, br, gzip, deflate]
- keep wildcardAcceptEncoding for Accept-Encoding: * limited to gzip and deflate
- update error message and directive regex to include zstd
- adjust tests to reflect new zstd preference and wildcard behavior
- clarify default behavior when no Accept-Encoding header is sent
- document zstd support and encoding priorities in the README
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0deefb8) to head (b9c814a).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #274   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines          217       219    +2     
  Branches        36        36           
=========================================
+ Hits           217       219    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Only run zstd-related encoding cases when zstd is supported
@uhop uhop self-assigned this Dec 2, 2025
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.

2 participants