Skip to content

docs: add 1.9.0 changelog entry - #21

Merged
rhoerr merged 1 commit into
mage-os:masterfrom
marcelmtz:docs/changelog-1.9.0
Aug 5, 2026
Merged

rhoerr merged 1 commit into
mage-os:masterfrom
marcelmtz:docs/changelog-1.9.0

Conversation

@marcelmtz

Copy link
Copy Markdown
Contributor

Summary

Adds the 1.9.0 entry to CHANGELOG.md for the template import hardening merged in cc520fd. The advisory merge pushes the fix commits to master but does not produce a changelog entry or a release, so this fills the first gap.

Why 1.9.0 and not 1.8.2

The fix carries three behavior changes, so a patch bump would be misleading:

  1. Template import now fails when an archive contains an asset that is not an allowed image type (jpg, jpeg, png, gif, webp). Previously any file type was copied into pub/media, so templates exported with e.g. SVG assets stop importing and need re-exporting.
  2. TemplateManagement::storePreviewImage() throws a LocalizedException on failure instead of returning null.
  3. CmsConverter::__construct() gains a PathValidator argument; TemplateManagement::__construct() gains PathValidator and LoggerInterface. Subclasses calling parent::__construct() positionally need updating.

All three are called out under ### Changed so integrators hit them in the changelog rather than in production.

Note on the GHSA link

The entry links the advisory, which is still in draft. The link resolves once the advisory is published — worth landing this together with the release and the publish rather than long before.

Remaining release steps

  • Merge this
  • Tag 1.9.0 and create the GitHub release
  • Verify Packagist picks up 1.9.0
  • Publish GHSA-hrj3-88v2-6wjx with patched version 1.9.0

🤖 Generated with Claude Code

https://claude.ai/code/session_014NRhJ9fkQuBzwbpXB5oi5b

Records the template import hardening merged in cc520fd and the three
behavior changes it introduces: non-image template assets are no longer
imported, storePreviewImage() throws instead of returning null, and
CmsConverter / TemplateManagement gained constructor arguments.

Minor rather than patch, since existing templates containing non-image
assets stop importing and the constructor signatures change.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014NRhJ9fkQuBzwbpXB5oi5b
@marcelmtz
marcelmtz requested review from a team and dadolun95 as code owners August 5, 2026 18:12
@rhoerr
rhoerr merged commit 794f48f into mage-os:master Aug 5, 2026
1 check failed
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