nextcloud / docker

⛴ Docker image of Nextcloud
https://hub.docker.com/_/nextcloud/
GNU Affero General Public License v3.0
5.94k stars 1.82k forks source link

Add support for brotli compression to the apache image #2076

Open st3iny opened 11 months ago

st3iny commented 11 months ago

First of all, thanks for maintaining this awesome docker image.

I have a feature request for the apache images: Would you be open to add support for brotli compression to facilitate more efficient bandwidth usage? If so, I'd like to start working on it and see if it is feasible.

Of course, I'd be happy to open a PR if it works well.

joshtrichards commented 10 months ago

I think the idea here (with the micro-services images) is to stay as close as possible to a conventional/recommended Nextcloud Server installation as possible. So I think a couple things will have to happen first:

Then (particularly those last two) could trickle through to changes in the Docker micro-service images here (for both apache and nginx).

See the AIO version referenced above for an Apache model. :smile:

kwinsch commented 1 month ago

If you have to much processing power, and really focus on high compression over throughput, you could use an external reverse proxy to do the job like described under https://github.com/google/ngx_brotli/issues/162

This approach even allows to compress gzip at higher compression levels, use brotli or the newer zstd.

I have not verified the compatibility with nextcloud, but it works in general.

st3iny commented 1 month ago

Yeah, this is what I do in production. My server has only a few active users and bandwidth is a bigger issue than processing power.