Skip to content

[8.0] Symfony 8 / PHP 8.5 modernisation#7

Merged
wtorsi merged 2 commits intomainfrom
8.0
Mar 13, 2026
Merged

[8.0] Symfony 8 / PHP 8.5 modernisation#7
wtorsi merged 2 commits intomainfrom
8.0

Conversation

@wtorsi
Copy link
Contributor

@wtorsi wtorsi commented Mar 2, 2026

Summary

  • Major refactor: PSR-4 src/ layout, S3 loader/resolver support, async Messenger processing, client SDK, security hardening
  • Quality & DX: ImageFormat/ImagineDriver enums, #[Override] attributes, driver short aliases (gd/imagick/gmagick), configurable binary paths
  • Security: SSRF protection on StreamLoader (URI scheme allowlist), DoS pixel budget on output dimensions, S3 dangerous extension blocking, Cache-Control/ACL on S3 uploads
  • Cache optimisation: resolveIfStored() single-pass existence check + URL resolution on all resolvers
  • Bug fixes: 14 PHPStan fixes, DI wiring fixes, density propagation, binary MIME fallback, absolute path handling
  • Twig: restored image_filter filter, added filter parameter to fit/fill/optimize, removed unused avif filter
  • Tests: comprehensive unit + integration test coverage for controllers, cache, security, concurrency, bundle boot, filter manager, processors, post-processors
  • Docs: README updated to reflect all changes — new config options, SSRF docs, S3 options, enum references

Test plan

  • composer test passes all unit and integration tests
  • composer cs-check reports no style violations
  • PHPStan analysis passes at max level
  • Verify Twig filters (image_filter, fit, fill, optimize) work in a Symfony 8 app
  • Verify S3 resolver uploads include Cache-Control header
  • Verify StreamLoader rejects disallowed URI schemes

🤖 Generated with Claude Code

Dev and others added 2 commits March 2, 2026 10:19
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…n Twig macros

Add path_style and credentials options to S3ResolverFactory for
LocalStack/MinIO compatibility. Update Twig image macros to prefer
relativePath over uri for correct image filter resolution with S3.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wtorsi wtorsi merged commit 99da8a9 into main Mar 13, 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.

1 participant