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

Switch from GZIP to Brotli by default #343

Closed
kytta opened this issue Nov 9, 2023 · 4 comments · Fixed by #346
Closed

Switch from GZIP to Brotli by default #343

kytta opened this issue Nov 9, 2023 · 4 comments · Fixed by #346
Assignees

Comments

@kytta
Copy link

kytta commented Nov 9, 2023

Brotli, as we know, is a more superior compression algorithm; fast & effective. According to Can I Use, 97% of devices can use Brotli. It's widely adopted by the servers, too: nginx, Apache, LiteSpeed, lighttpd, Caddy, and others. Big CDNs like Fastly and Cloudflare are enabling it by default.

Wouldn't it make sense for size-limit to flip the switch, too, and show the "brotlied" result by default, making GZIP opt-in and encouraging people to switch their infrastructure?

@ai
Copy link
Owner

ai commented Nov 10, 2023

Sounds like a good idea. But I will ask in Twitter first to collect the feedback.

@ai
Copy link
Owner

ai commented Nov 11, 2023

Here is PR #346

@ai ai self-assigned this Nov 11, 2023
@ai
Copy link
Owner

ai commented Nov 11, 2023

Here is a good statistics about the real web traffic in 2022 https://almanac.httparchive.org/en/2022/cdn#brotli-adoption

  • 47% of CDN traffic is Brotli
  • 25% of self-hosted servers

I think it is pretty solid numbers. In 2023, they will be better. And this is the whole traffic. Size Limit users is a little more advanced.

Screenshot from 2023-11-11 07-11-26

@ai ai closed this as completed in #346 Nov 13, 2023
@ai
Copy link
Owner

ai commented Nov 13, 2023

Released in 11.0.

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 a pull request may close this issue.

2 participants