Skip to content

[fix] five runtime bugs: density propagation, binary fallback, absolu…#6

Merged
wtorsi merged 1 commit intomainfrom
8.0
Feb 27, 2026
Merged

[fix] five runtime bugs: density propagation, binary fallback, absolu…#6
wtorsi merged 1 commit intomainfrom
8.0

Conversation

@baldrys-ed
Copy link
Contributor

…te paths, MIME guesser, S3 config

  • CacheManager: is_int → is_numeric in extractDensity (HTTP params are strings)
  • ImageRuntime: propagate fit/fill/optimize options into processor via array_replace
  • FileSystemLocator: strip leading slash instead of throwing (file-bundle paths start with /)
  • PostProcessors: return original binary when binary is not executable
  • BinaryMimeTypeGuesser: remove MimeTypeGuesserInterface (class works with binary content, not file paths)
  • S3ResolverFactory: bucket isRequired() → defaultNull() (was applied to all resolver types)
  • Tests: update PostProcessor tests to reflect graceful fallback behavior
  • Tests: add coverage for string density from HTTP params, leading slash, density propagation

…te paths, MIME guesser, S3 config

- CacheManager: is_int → is_numeric in extractDensity (HTTP params are strings)
- ImageRuntime: propagate fit/fill/optimize options into processor via array_replace
- FileSystemLocator: strip leading slash instead of throwing (file-bundle paths start with /)
- PostProcessors: return original binary when binary is not executable
- BinaryMimeTypeGuesser: remove MimeTypeGuesserInterface (class works with binary content, not file paths)
- S3ResolverFactory: bucket isRequired() → defaultNull() (was applied to all resolver types)
- Tests: update PostProcessor tests to reflect graceful fallback behavior
- Tests: add coverage for string density from HTTP params, leading slash, density propagation
@wtorsi wtorsi merged commit 28af14e into main Feb 27, 2026
1 check passed
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.

2 participants