Closed Bug 634226 Opened 14 years ago Closed 14 years ago

[amo] Add . to cdn file extension regex

Categories

(Infrastructure & Operations Graveyard :: WebOps: Other, task)

task
Not set
critical

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: standard8, Assigned: oremj)

Details

(Whiteboard: [gs])

This was just reported on Thunderbird's support site (http://gsfn.us/t/23x32): === Start Quote === I want to download a Spanish dictionary. What should I do with this response? "Forbidden You don't have permission to access /firefox/downloads/latest/7020/addon-7020-latest.xpi on this server." Thanks! === End Quote === Looking at the link from the dictionaries page for the Mexican one: https://addons.mozilla.org/en-US/thunderbird/addon/diccionario-espa%C3%B1ol-m%C3%A9xico/ this gets redirected to: https://static.addons.mozilla.net/en-US/thunderbird/addon/diccionario-espa%C3%B1ol-m%C3%A9xico/ The majority of links on that static page are then 404.
(In reply to comment #0) > The majority of links on that static page are then 404. Including the download link, however, if I change the download link from: https://addons.mozilla.org/thunderbird/downloads/latest/7020/addon-7020-latest.xpi?src=addondetail to: https://static.addons.mozilla.net/thunderbird/downloads/latest/7020/addon-7020-latest.xpi?src=addondetail the download works again.
(In reply to comment #1) > Including the download link, however, if I change the download link from: > > https://addons.mozilla.org/thunderbird/downloads/latest/7020/addon-7020-latest.xpi?src=addondetail > > to: > > https://static.addons.mozilla.net/thunderbird/downloads/latest/7020/addon-7020-latest.xpi?src=addondetail Oops, sorry, swap those two around ;-)
Bug 628774 added redirects for static files and bug 631097 added .ico to the list. We need to prefix those matches with a \. so that ico$ does not match amo/addon/mexico.
Assignee: nobody → server-ops
Component: Administration → Server Operations: Web Content Push
Product: addons.mozilla.org → mozilla.org
QA Contact: administration → mrz
Summary: Spanish (Mexican) dictionary downloads are broken → [amo] Add . to cdn file extension regex
Version: unspecified → other
Assignee: server-ops → jeremy.orem+bugs
I've pushed this to preview and next. Can you verify?
Pushed to production.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Verified, thanks.
Status: RESOLVED → VERIFIED
Component: Server Operations: Web Operations → WebOps: Other
Product: mozilla.org → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.