Needs review
Project:
Cloudinary Integration
Version:
3.0.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
9 Oct 2024 at 13:35 UTC
Updated:
13 Nov 2024 at 16:48 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
klaasvw commentedThe attached patch is a start for fixing the above issues. Feedback and improvements are definitely welcome!
Comment #4
klaasvw commentedThe following patch applies the asset caching like the previous patch but also caches Admin API calls in the cache bin. Becasue Cloudinary has a 500 to 2000 requests / hour limit on the admin API and this module calls the API multiple times for every asset it loads the calls add up quickly.
Comment #10
klaasvw commentedUpdated the MR and added a patch that also caches admin API calls from the cloudinary_stream_wrapper module. Also implements a fix from #3354894: Cloudinary Stream Wrapper caches items in the database (Database Storage) but never actually checks if an item is inside the storage so the db storage is not bypassed.
Comment #11
klaasvw commented