Skip to content

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