983815 - Use CDN for homepage/category/search API responses

Status: RESOLVED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P1)

Description

[Christopher Van Wiemeersch [:cvan]]

We should use the CDN for the following API responses:

https://marketplace.firefox.com/api/v\d+/fireplace/search/featured/.*
https://marketplace.firefox.com/api/v\d+/apps/category/.*
https://marketplace.firefox.com/api/v\d+/apps/search/.*

These are API responses that contain no user information and are loaded from the homepage, category landing, and search results pages, respectively.

Bug 957387 was to investigate whether we can do this. And we can, even with async cache refreshing, but this is blocked on Akamai at the moment.

When Akamai gets the ball rolling, we can get the ball rolling on this. I'm excited!

Comment 1

[Mathieu Pillard [:mat]]

You can replace https://marketplace.firefox.com/api/v\d+/apps/search/.* with https://marketplace.firefox.com/api/v\d+/fireplace/search/.* - (See bug 978059 and bug 982992)

Comment 2

[Christopher Van Wiemeersch [:cvan]]

https://github.com/mozilla/fireplace/commit/076871a

Jeremy set up on Cloudflare on -dev, and I hooked up our search, category, and featured (homepage) API responses to it :)

On fast wifi:
http://f.cl.ly/items/0I1G2B1X112i2D2P2L0T/Screen%20Shot%202014-03-27%20at%205.37.56%20PM.png

WOOHOOOOO!!!

Sub-EDGE (48K):
http://f.cl.ly/items/043P2F3J2c0H2Z1r3I2X/Screen%20Shot%202014-03-27%20at%205.41.39%20PM.png

Comment 3

[Mathieu Pillard [:mat]]

When loading https://marketplace-dev.allizom.org/ I see it's requesting https://marketplace-dev.mozflare.net/ for assets but it's using https://marketplace-dev-cdn.allizom.org/ for the search/featured XHR request:

https://marketplace-dev-cdn.allizom.org/api/v1/fireplace/search/featured/?cache=1&cat=&lang=en-US&region=restofworld&vary=0

In console:

>>> require('settings').cdn_url
"https://marketplace-dev-cdn.allizom.org"

>>>require('settings').media_url
" https://marketplace-dev-cdn.allizom.org/media"

(Note the space in media_url, and the fact that it's not using mozflare in both)


I saw this in both Firefox and Chrome.

Comment 4

[Christopher Van Wiemeersch [:cvan]]

Fixed, thanks: https://github.com/mozilla/fireplace/commit/9af76d3

Comment 5

[Iulian Timis]

Please add STR here or mark it with [qa-] if no QA is needed.