decentraland / decentraland/asset-bundle-converter

Evaluate if Brotli file variants are still needed

Open
#260 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
4
Forks
8
Avg merge
26m
Merged PRs (30d)
1

Description

## Context

During the upload step in `consumer-server/src/logic/conversion-task.ts`, all converted asset bundles are uploaded with both `Brotli` and `Uncompressed` variants. This doubles the number of files stored in S3.

## Question

Is the Brotli variant still needed? Cloudflare and most CDNs can handle compression at the edge, so pre-compressing and storing `.br` files may be redundant overhead in both storage and conversion time.

## Action items
- [ ] Check if the explorer client (WebGL/desktop) explicitly requests `.br` files or relies on CDN `Accept-Encoding` negotiation
- [ ] Measure storage cost impact of removing Brotli variants
- [ ] If not needed, remove `FileVariant.Brotli` from the upload matches and clean up related code

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.