- Issue created by @rp7
- Merge request !9Issue #3444326: Don't perform redirects on asset generation paths → (Open) created by rp7
- Issue was unassigned.
- Status changed to Needs review
7 months ago 9:55am 30 April 2024 - Status changed to Needs work
7 months ago 12:02pm 30 April 2024 - 🇦🇺Australia mstrelan
Left some nits and a question about compatibility. Test coverage would be good if possible.
- 🇧🇪Belgium rp7
Made a few proposed changes + tried to get the tests green.
A test for an actual asset generation path should still be added, though. - Status changed to Needs review
5 months ago 4:06am 17 June 2024 - 🇦🇺Australia mstrelan
Rebased and started looking at test coverage, but one glaring issue is that the asset generation directory by default is the same as the public file system. This might be ok though, since it's usually served directly by apache anyway, except for the image styles directory.