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

Add support for Zstd compressed static files #52

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Add support for Zstd compressed static files #52

merged 1 commit into from
Mar 13, 2024

Conversation

kescherCode
Copy link
Contributor

Allow serving Zstd compressed files with the .zst extension.

The example has not been adjusted yet, because the SDK's supported CompressionFormats do not yet include zstd. However, static files compressed by other means (external tools) still work.

Rationale:

Chrome 123 added zstd Content-Encoding support behind a feature flag enable-zstd-content-encoding, and Mozilla has committed to implementing it in Firefox as well.

Allow serving Zstd compressed files with the `.zst` extension.

The example has not been adjusted yet, because the SDK's supported CompressionFormats do not yet include zstd. However, static files compressed by other means (external tools) still work.

Rationale:

[Chrome 123 added zstd Content-Encoding support behind a feature flag `enable-zstd-content-encoding`](https://chromestatus.com/feature/6186023867908096), and [Mozilla has committed to implementing it in Firefox as well](mozilla/standards-positions#775).
Copy link
Member

@MarkStega MarkStega left a comment

Choose a reason for hiding this comment

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

Looks good; I will create a release after I handle the dependabot updates that are needed.

Thanks for the contribution.

@MarkStega MarkStega merged commit c15055f into Material-Blazor:main Mar 13, 2024
1 check passed
@MarkStega
Copy link
Member

@kescherCode

Jeremy, thanks again; I just pushed the 2.1.0 release, it will take a couple of minutes for nuget to show the new version.

@simonziegler
Copy link
Member

Hey @kescherCode, many thanks for this contribution!

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.

3 participants