1218626 - Wrap Optimize.ly tags on www.moz.org in DNT conditional

Status: RESOLVED FIXED

(www.mozilla.org :: Analytics, defect)

Description

[Javaun Moradi [:javaun]]

We're wrapping Google Analytics in a script to check for user DNT intent, otherwise we will load GA on the page. GA is on the Tracking Protection blocklist under the "analytics" section, so all pages with GA trigger the shield for users running TP.

Optimize.ly is very likely to be reclassified as an analytics tracker in the coming weeks and like GA will trigger the TP shield. 

We should wrap Optimize.ly in a DNT conditional similar to what we're doing for GA in bug 1217896

Comment 1

[Jennifer Bertsch [:jbertsch]]

Hi Schalk-

Do you think you can get to this one in the next month or so?

Thx,
Jen

Comment 2

[Schalk Neethling [:espressive]]

(In reply to Jennifer Bertsch [:jbertsch] from comment #1)
> Hi Schalk-
> 
> Do you think you can get to this one in the next month or so?
> 
> Thx,
> Jen

Yup, will add it to my to-do list.

Comment 3

[Jonathan Petto [:jpetto]]

:espressive - Were you still working this one? Would be happy to take it on if you're busy. :)

Comment 4

[Schalk Neethling [:espressive]]

(In reply to Jon Petto [:jpetto] from comment #3)
> :espressive - Were you still working this one? Would be happy to take it on
> if you're busy. :)

Hey Jon, feel free to grab it. Thanks!

Comment 5

[Jonathan Petto [:jpetto]]

Attachment: GitHub PR

Comment 6

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/5a96be584389a708bbbbcb69f5e03d7a100745a8
[fix bug 1218626] Wrap Optimizely script in DNT check.

https://github.com/mozilla/bedrock/commit/241d55c0a910f27f1de7c30ebbf2b19ef64522cc
Merge pull request #4735 from jpetto/bug-1218626-wrap-optimizely-in-dnt

[fix bug 1218626] Wrap Optimizely script in DNT check.