Closed Bug 566539 Opened 14 years ago Closed 14 years ago

Migrate /download/latest/* to zamboni

Categories

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

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 591028
4.x (triaged)

People

(Reporter: clouserw, Unassigned)

Details

We're getting a lot of broken add-on downloads these days, and I have no idea why, but since the page is up for migration anyway, I'd like to just get it on zamboni.  I don't see myself getting to this so I'm filing a much-hated nobody@ bug that we'll bump if we have to. :-/

The remora latest() code is in controllers/downloads_controller.php and is only about 25 lines so this shouldn't be too tough.  It does need tests though since we're dealing with platforms and type (jar vs xpi).  The function claims it supports: /downloads/latest/$id/$type/$platform/ URLs, but I grepped through a week of logs and never found type (I did find platform).  So we should see what is going on there.

Also, that URL is fugly, I think we should do something like /addon/$id/download/latest/ in zamboni.  Note that we need to have a "filename.ext" on the end of the URL like we do with remora.  We could 302 a prettier URL to the actual file download URL, but I'd want that to be pretty heavily cached, so I don't know if that would be a great idea.  Also, it would mean writing that function in zamboni too.
Target Milestone: 5.11 → 5.12
Target Milestone: 5.12 → 4.x (triaged)
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.