Skip to content

Remove Brotli dependency #259

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

Merged
merged 1 commit into from
Jun 17, 2025
Merged

Remove Brotli dependency #259

merged 1 commit into from
Jun 17, 2025

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Jun 17, 2025

Since the Brotli dependency adds ~7-8MB (depending on arch) to the app image (along with having to download/install the wheel during the build), and isn't currently used by the Getting Started guide, since it doesn't have any static assets that are compressible.

$ ls -alh /layers/heroku_python/venv/lib/python3.13/site-packages/
...
-rwxr-xr-x  1 heroku heroku 7.2M Jan  1  1980 _brotli.cpython-313-aarch64-linux-gnu.so
$ ls -al /workspace/staticfiles/ 
total 16
drwxr-xr-x 2 heroku heroku 4096 Jan  1  1980 .
drwxrwxrwx 5 heroku heroku 4096 Jan  1  1980 ..
-rw-r--r-- 1 heroku heroku 2217 Jan  1  1980 lang-logo.019c8743b7cf.png
-rw-r--r-- 1 heroku heroku  100 Jan  1  1980 staticfiles.json

See:
https://whitenoise.readthedocs.io/en/latest/django.html#brotli-compression

GUS-W-18705397.

Since the Brotli dependency adds ~8MB to the app image (along with
having to download/install the wheel during the build), and isn't
currently used by the Getting Started guide, since it doesn't have
any static assets that are compressible.

```
-rwxr-xr-x  1000:1000   7.5 MB       │      ├── _brotli.cpython-313-aarch64-linux-gnu.so
```

See:
https://whitenoise.readthedocs.io/en/latest/django.html#brotli-compression

GUS-W-18705397.
@edmorley edmorley self-assigned this Jun 17, 2025
@edmorley edmorley requested a review from a team as a code owner June 17, 2025 12:27
@edmorley edmorley removed the request for review from a team June 17, 2025 12:27
@heroku heroku bot temporarily deployed to getting-star-edmorley-r-qqgch3 June 17, 2025 12:27 Inactive
@edmorley edmorley merged commit aa8ad15 into main Jun 17, 2025
1 check passed
@edmorley edmorley deleted the edmorley/rm-brotli branch June 17, 2025 12:36
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