- π«π·France duaelfr Montpellier, France
Thanks a lot!
#2 is right, it works with MR from π Rendering duplicate images when the original images have the same name but different extension RTBCInstead of asking the browser to reload, wouldn't it be faster and safer to send the response with the file content?
I wonder how this would work with some kind of reverse proxy trying to cache responses. - πΊπΈUnited States nicxvan
This does not work for me, important note the patch mentioned here does not apply for me: https://www.drupal.org/project/webp/issues/3328040#comment-14830291 π Review webp filename logic Closed: duplicate
I tried https://www.drupal.org/project/webp/issues/3281606 π Rendering duplicate images when the original images have the same name but different extension RTBC instead because it is a duplicate, but it still did not work.
- π«π·France duaelfr Montpellier, France
- πΊπΈUnited States nicxvan
Sorry I wasn't clear, the patch in #2 does not apply for me, I was going to update the other ticket and patch so it applies cleanly.
- π©πͺGermany berliner
Found an issue with the patch. The ProxySubscriber redirected to a non-valid URL. The attached patch fixes that.
- First commit to issue fork.
- Merge request !35Issue #3325808: Webp Image not generated for styles using original image β (Open) created by jrglasgow
- πΊπΈUnited States jrglasgow Idaho
I have created an issue fork for the patch in #11
- π©πͺGermany berliner
I have rebased the MR and made an additional change not redirect to the same requested URL if the webp creation succeeded. Only redirect to source image if the webp creation failed. Also did some cleanup of the comments and the method name in the ProxySubscriber to better reflect what it actually does.