Feature Request md5 in file instead of parameter

11 views
Skip to first unread message

Bigb

unread,
Jul 19, 2009, 12:12:01 PM7/19/09
to SmartSprites CSS Sprite Generator Users and Developers
Hello,

I play with smartsprite for some hours and it's exactly what i've
searched. Works perfectly for me with ant integration.

I have just one request about sprite name : could you add an option to
add md5 in sprite name instead of a parameter. Because many proxy will
not cache the sprite because of this, Yahoo performance team
recommends to put md5 directly in filename. For example where i worked
at AirFrance, the proxy doesn't cache images with parameter.

Thanks for the tool.

Nicolas

Stanislaw Osinski

unread,
Jul 19, 2009, 2:17:01 PM7/19/09
to smartsprites-css...@googlegroups.com
Hi Nicolas,

I play with smartsprite for some hours and it's exactly what i've
searched. Works perfectly for me with ant integration.

I'm glad SmartSprites works for you, can I quote the above on the website? :-)

I have just one request about sprite name : could you add an option to
add md5 in sprite name instead of a parameter. Because many proxy will
not cache the sprite because of this, Yahoo performance team
recommends to put md5 directly in filename. For example where i worked
at AirFrance, the proxy doesn't cache images with parameter.

Very good point, Nicolas. I've created a ticket for that change, I should be able to push the change in the next minor version (hopefully next week when I'm back from my short holiday):

http://issues.carrot2.org/browse/SMARTSPRITES-41

As a heads up: as described in the "Discussion" section of the above ticket, the previous behaviour of the sprite-image-uid directive will change (always embedding hash in sprite image file name). Does that seem like an issue for anyone?

Thanks,

S.


redclover

unread,
Jul 20, 2009, 9:06:39 AM7/20/09
to SmartSprites CSS Sprite Generator Users and Developers
I haven't investigated this issue that far but I think it will no
problem at all in our application. The sprite image is dynamically
generated itself so if it is named xy or xyz will make no
difference :-) In fact we did it that way (but using just a timestamp
instead of the md5 hash) before smartsprite itself provided this
feature.

BigKev

unread,
Jul 21, 2009, 3:10:02 AM7/21/09
to SmartSprites CSS Sprite Generator Users and Developers
Well it works very well in our deployment and we have caching servers
- so if implementing that feature, could have have an option to choose
between the two methods?

Bigb

unread,
Jul 22, 2009, 10:37:14 AM7/22/09
to SmartSprites CSS Sprite Generator Users and Developers
Thanks for your support. For me there is no problem since the
build.xml file is not finished yet.


On Jul 19, 8:17 pm, Stanislaw Osinski <stac...@gmail.com> wrote:
Reply all
Reply to author
Forward
0 new messages