Fix dns prefetch for google analytics

RESOLVED FIXED in 2015-03-17

Status

Marketplace
Consumer Pages
P3
trivial
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mat, Assigned: ngoke)

Tracking

({perf})

2015-03-17
Points:
---

Details

(Whiteboard: [goodfirstbug])

(Reporter)

Description

3 years ago
In fireplace's src/app.html we have:
<link rel="dns-prefetch" href="//ssl.google-analytics.com">

This is good, except... :
- tracking.js uses https://www.google-analytics.com/analytics.js
- that script uses https://www.google-analytics.com/analytics and https://stats.g.doubleclick.net/
- zamboni's mkt/commonplace/templates/commonplace/index.html, which is the template actually used in production, only sets <link rel="dns-prefetch" href="{{ media_origin }}">


dns-prefetching our media origin is probably useless (the other <link> elements in our <head> already references that origin), so we should fix zamboni to properly prefetch //www.google-analytics.com and maybe //stats.g.doubleclick.net as well. It's trivial but could help perf on mobile networks, where latency is killer.

Updated

3 years ago
Priority: -- → P3
Whiteboard: [goodfirstbug]
Unsure if this is still relevant (probably), but adding to UA tracker.
Assignee: nobody → kngo

Updated

3 years ago
Blocks: 1134407
(Assignee)

Comment 2

3 years ago
https://github.com/mozilla/zamboni/pull/2975
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-03-17

Comment 3

3 years ago
Please add some STRs to this bug or mark it as [qa-]
You need to log in before you can comment on or make changes to this bug.