Closed Bug 649789 Opened 14 years ago Closed 14 years ago

Parts of the file viewer being rewritten to the cdn

Categories

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

Product:
addons.mozilla.org Graveyard
Component:
Developer Pages
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: andy+bugzilla, Assigned: oremj)

References

Details

File viewer has the filename in the url. The filename could end with .js, and seems to get rewritten onto the CDN. For example: https://addons.allizom.org/z/en-US/firefox/files/browse/115275/file/defaults/preferences/yslow.js Becomes: https://addons-cdn.allizom.org/z/en-US/firefox/files/browse/115275/file/defaults/preferences/yslow.js Either we can a) stop using the filename in the file viewer (although I kind of like it) or b) white list /files/browse/ and /files/compare/ so they don't.
Blocks: 645720
After chatting to clouserw can we go with item b) please.
Assignee: nobody → jeremy.orem+bugs
Did this in preview/next: - RewriteCond %1 \.(css|gif|ico|jpg|js|jsi18n|png|svg|woff)$ + RewriteCond %1 \.(?:css|gif|ico|jpg|js|jsi18n|png|svg|woff)$ + RewriteCond %1 !/files/(?:browse|compare)/ Can someone try it out?
Don't think so, looks like the URL is still being rewritten.
Should be fixed now.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Looks good, thanks.
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.