feat: use brotli instead of gzip for compression in NGINX #1593
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Azure Demo Servers are available: |
3 similar comments
|
Azure Demo Servers are available: |
|
Azure Demo Servers are available: |
|
Azure Demo Servers are available: |
andreassteinmann
force-pushed
the
explore/brotli
branch
from
f1cd8ba to
a11a77b
Compare
|
Azure Demo Servers are available: |
4 similar comments
|
Azure Demo Servers are available: |
|
Azure Demo Servers are available: |
|
Azure Demo Servers are available: |
|
Azure Demo Servers are available: |
andreassteinmann
force-pushed
the
explore/brotli
branch
from
61dc233 to
1367d1b
Compare
shauke
changed the title
shauke
changed the title
…ke && make install instead of make modules
…e modules instead of make install
…ration but without --add-module
andreassteinmann
force-pushed
the
explore/brotli
branch
from
c0699e6 to
e8a04bc
Compare
|
Azure Demo Servers are available: |
Closed
shauke
previously approved these changes
Mar 15, 2024
|
@intershop/c3po-documentation-l10n can you please review the documentation parts, since a bigger part documentation was added |
mglatter
reviewed
Mar 18, 2024
mglatter
reviewed
Mar 19, 2024
mglatter
previously approved these changes
Mar 19, 2024
Co-authored-by: MGlatter <m.glatter@intershop.de>
shauke
approved these changes
Mar 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Type
[x] Feature
What Is the Current Behavior?
gzip is used to compress files on NGINX.
What Is the New Behavior?
ngx_brotli is used to compress files on NGINX because Brotli has a better compression ratio compared to gzip.
Two new NGINX modules ngx_http_brotli_filter_module.so (compressing responses on-the-fly) and ngx_http_brotli_static_module.so (serving pre-compressed files) are built in the Dockerfile and referenced in nginx.conf.
Does this PR Introduce a Breaking Change?
[x] No
Other Information
AB#94548