Closed (fixed)
Project:
Cache Expiration
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Mar 2011 at 07:37 UTC
Updated:
11 Jul 2013 at 07:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
mikeytown2 commentedMy thoughts:
- Aggregated js/css files is taken care of by http://drupal.org/project/advagg - One could make a sub module to handle all css/js files; even those not aggregated by placing a single file into its own bundle.
- Image derivatives could be done with this http://groups.drupal.org/node/127079
- File api - doesn't it append _0 to the file if you upload the same filename?
- profile images... there's a reason this is not in D7 core.
But looking over that issue and expire seems to be the right place for it. Go for it!
You might be able to get some pointers from my other module for caching image info
http://drupal.org/project/imageinfo_cache
Comment #2
mikeytown2 commentedI would make it a sub module
Comment #3
acbramley commentedBumping this to 7.x-1.x as this would be great for expiring file urls when (for example) you update a file using the Media module. I have run into issues with this on a production site where clients were updating files and the old file was staying cached in Varnish for a long time. I'm going to begin looking into this now.
Comment #4
acbramley commentedThis is an untested stab at this functionality, will report with testing outcome shortly.
Comment #5
acbramley commentedAbove patch doesn't work as I had the base path in the url being passed in. This new patch seems to work well, I've tested updating a file and see it correctly being purged from Varnish.
Comment #6
acbramley commentedComment #7
spleshkaPlease, re-roll your patch and provide some UI settings for 7.x-2.x branch. I will glandy apply it.
Comment #8
acbramley commentedAwesome, I like where the new branch is going, much nicer structure :) Here's the patch
Comment #9
acbramley commentedWoops, patch didn't include .inc file
Comment #10
spleshkaGreat! I've made some minor changed in the patch and commited/pushed it to 7.x-2.x. Thanks a lot for patch.