github / advisory-database

Security vulnerability database inclusive of CVEs and GitHub originated security advisories from the world of open source software.
Creative Commons Attribution 4.0 International
1.72k stars 323 forks source link

[GHSA-qjqp-xr96-cj99] Trix Editor Arbitrary Code Execution Vulnerability #4482

Closed chadlwilson closed 4 months ago

chadlwilson commented 4 months ago

Updates

Comments Suggesting adding the actiontext component of Rails versions 7.0 and 7.1 as documented as being affected by the Rails team at https://discuss.rubyonrails.org/t/xss-vulnerabilities-in-trix-editor/85803 https://rubyonrails.org/2024/5/17/Rails-Versions-7-0-8-3-has-been-released https://rubyonrails.org/2024/5/17/Rails-Versions-7-0-8-2-and-7-1-3-3-have-been-released

Not sure if this is the appropriate way to manage these situations where one product embeds another, but seems reasonable.

github commented 4 months ago

Hi there @afcapel! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

advisory-database[bot] commented 4 months ago

Hi @chadlwilson! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

afcapel commented 3 months ago

@chadlwilson makes sense to me, since ActionText bundles Trix dist files 👍

But I'm afraid I don't maintain Trix anymore and I can't help here.

chadlwilson commented 3 months ago

I'm not sure how this works with the GH Reviewed Advisories, but somehow this already got merged by the bot (someone?) and applied to the main advisory anyway... for better or worse!

chadlwilson commented 3 months ago

Probably reviewed by core team, and more of an "FYI" to Trix folks in case one wanted to also make changes to the repo-specific advisory.