Skip to content

compress plugin - use configured compression levels for gzip and brotli#12924

Open
JakeChampion wants to merge 1 commit intoapache:masterfrom
JakeChampion:jake/master-c1
Open

compress plugin - use configured compression levels for gzip and brotli#12924
JakeChampion wants to merge 1 commit intoapache:masterfrom
JakeChampion:jake/master-c1

Conversation

@JakeChampion
Copy link
Contributor

Previously, gzip and brotli compression used hardcoded levels even when custom levels were configured via HostConfiguration

Split initialization into data_alloc() and transform_init() so

  • gzip uses hc->zlib_compression_level()
  • brotli uses hc->brotli_compression_level() and hc->brotli_lgw_size()

Also adds proper error handling if encoder initialization fails

@JakeChampion JakeChampion changed the title Use configured compression levels for gzip and brotli compress plugin - use configured compression levels for gzip and brotli Feb 27, 2026
Previously, gzip and brotli compression used hardcoded levels even when custom levels were configured via `HostConfiguration`

Split initialization into `data_alloc()` and
`transform_init()` so
that gzip uses `hc->zlib_compression_level()`, and brotli uses
`hc->brotli_compression_level()` and `hc->brotli_lgw_size()`

Also adds proper error handling if encoder initialization fails
@maskit maskit added the compress compress plugin label Feb 27, 2026
@maskit maskit added this to the 11.0.0 milestone Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compress compress plugin

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants