731097 - Move collection icon logic out of apache and into python

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Wil Clouser [:clouserw]]

Read bug 731089 for background and some info on doing this exact thing for add-on icons and for the already-done user images.  I think the rewrite rules here are:

>    RewriteRule ^([^/]+/[^/]+/|)images/collection_icon/((\d*?)\d{1,3}).png /img/uploads/collection_icons/$3/$2.png [L]

>    RewriteCond %{REQUEST_FILENAME} !-f
>    RewriteCond %{REQUEST_FILENAME} /img/uploads/collection_icons/+\d*.png$
>    RewriteRule (\d+)\.png$ /img/uploads/collection_icons/0/$1.png [L]

Comment 1

[Wil Clouser [:clouserw]]

This will be turning this:

  https://addons-dev-cdn.allizom.org/en-US/firefox/images/collection_icon/2174527.png

into this:

  https://addons-dev-cdn.allizom.org/img/uploads/collection_icons/2174/2174527.png

Comment 2

[Wil Clouser [:clouserw]]

https://github.com/clouserw/zamboni/commit/98c27e043cb8fdb4597f5b35dbcc06bdfc8af3ec

QA: shouldn't notice much difference, just the URLs in comment 1

Comment 3

[krupa raj[:krupa]]

looks good. Verified uploading collections icons at https://addons-dev.allizom.org/en-US/firefox/collections/krupakrupa/collection989/