Support for expiring managed files on update [#1082860]

Idea taken from #1075526: Upstream CDN support?:
Would be usefull to have some way to expire URLs of static files like:
- profile images
- file api files (would include image fields)
- image derivatives like thumbnails etc
- javascript/css files
This would make expire more suitable to work with a cdn like cloudfront, akamai etc.

Comment	File	Size	Author
#9	1082860-expire-file-url-4.patch	6.3 KB	acbramley
#8	1082860-expire-file-url-3.patch	4.21 KB	acbramley
#5	1082860-expire-file-url-2.patch	1.03 KB	acbramley
#4	1082860-expire-file-url-1.patch	956 bytes	acbramley

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mikeytown2 CreditAttribution: mikeytown2 commented 6 March 2011 at 12:03

My 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 CreditAttribution: mikeytown2 commented 6 March 2011 at 18:57

I would make it a sub module

Comment #3

acbramley CreditAttribution: acbramley commented 30 April 2013 at 01:20

Version:

6.x-1.x-dev

» 7.x-1.x-dev

Bumping 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 CreditAttribution: acbramley commented 30 April 2013 at 02:02

Status:

Active

» Needs review

File	Size
1082860-expire-file-url-1.patch	956 bytes

This is an untested stab at this functionality, will report with testing outcome shortly.

Comment #5

acbramley CreditAttribution: acbramley commented 30 April 2013 at 02:33

File	Size
1082860-expire-file-url-2.patch	1.03 KB

Above 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.