Skip to content

Add CVE-2025-47964 for Symfony UX Twig/Live Component#751

Merged
stof merged 2 commits intoFriendsOfPHP:masterfrom
smnandre:cve-2025-47964
Aug 29, 2025
Merged

Add CVE-2025-47964 for Symfony UX Twig/Live Component#751
stof merged 2 commits intoFriendsOfPHP:masterfrom
smnandre:cve-2025-47964

Conversation

@smnandre
Copy link
Contributor

2.x:
time: 2025-05-19 12:05:00
versions: ['<2.25.1']
reference: composer://symfony/ux-live-component
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is ux-live-component actually impacted directly or only through its dependency on ux-twig-component (which would be reported anyway through the other advisory) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say directly as attribute were created and rendered directly from ux-live-component... no ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what happens if you use symfony/ux-live-component 2.24 with symfony/ux-twig-component 2.25.1 ? Would we still have the security issue, or would be it solved by the patch in symfony/ux-twig-component 2.25.1 ? That's the main question here (if updating only symfony/ux-twig-component to 2.25.1 removes the security vulnerability, we don't need an advisory for symfony/ux-live-component).

The fact that symfony/ux-live-component might be buggy (due to some double-escaping in places where it was working around the TwigComponent vulnerability before) when installing it alongside the patched ux-twig-component is a different topic (as this is not about a security issue)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ymfony/ux-live-component 2.24 with symfony/ux-twig-component 2.25.1

It would crash in multiple LiveComponent places, as ComponentAttributes would not have its second required argument when instancied (in live compoonent code)

@stof stof merged commit 63bf4d0 into FriendsOfPHP:master Aug 29, 2025
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