Skip to content

Tests: image_filter_max_pixels directive - #84

Draft
VadimZhestikov wants to merge 1 commit into
nginx:masterfrom
VadimZhestikov:image-filter-max-pixels
Draft

Tests: image_filter_max_pixels directive#84
VadimZhestikov wants to merge 1 commit into
nginx:masterfrom
VadimZhestikov:image-filter-max-pixels

Conversation

@VadimZhestikov

Copy link
Copy Markdown
Contributor

Adds image_filter_max_pixels.t, covering the new image_filter_max_pixels
directive: a source image with more pixels than the configured limit is
rejected, an image within the limit is processed, and a zero limit disables
the check. Version-guarded with try_run so it skips on builds without the
directive.

Companion to the ngx_http_image_filter_module source change (nginx/nginx#1575).

Verifies that source images with more pixels than the configured limit
are rejected, that images within the limit are processed, and that a
zero limit disables the check.

Also verifies that a source whose true dimensions exceed 16 bits is
measured in full: a 65540 x 2 PNG must be rejected, not understated to
4 x 2 by reading only the low 16 bits of the 32-bit PNG size fields.
@VadimZhestikov
VadimZhestikov force-pushed the image-filter-max-pixels branch from 01250a3 to c553198 Compare July 17, 2026 21:08
@VadimZhestikov
VadimZhestikov marked this pull request as draft July 21, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

2 participants