Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Add DNT header (Do Not Track) #446

Closed
ahunt opened this issue Apr 12, 2017 · 14 comments
Closed

Add DNT header (Do Not Track) #446

ahunt opened this issue Apr 12, 2017 · 14 comments
Labels
eng:automation Build automation, Continuous integration, .. feature QAApproved testing meta label for all features/bugs requiring test
Milestone

Comments

@ahunt
Copy link
Contributor

ahunt commented Apr 12, 2017

I think this is in the "expanded" potential feature list. I believe iOS already does this, and as a privacy focused browser I'm naively assuming we should probably be doing this - I'm therefore filing this now just to make sure we don't forget to make a decision.

@pocmo
Copy link
Contributor

pocmo commented Apr 19, 2017

From the feedback we got DNT is basically useless and we won't add it.

@pocmo pocmo closed this as completed Apr 19, 2017
@batisteo
Copy link

I thought it was a Mozilla initiative. I haven't seen anywhere it's deprecated.

@Rudloff
Copy link

Rudloff commented Jun 20, 2017

DNT is supported by several analytics tools like Piwik, it totally makes sense to enable it in Focus.

@comzeradd
Copy link

Many websites respect DNT, including our own (Mozilla) web properties. It makes sense for a browser like Focus to have this enabled by default.

@pocmo
Copy link
Contributor

pocmo commented Jun 26, 2017

Re-opening and flagging for triage.

@pocmo
Copy link
Contributor

pocmo commented Jul 12, 2017

I want to bring this up for two reasons:

  1. Some websites honor DNT and do not add tracking code if the browser adds those (e.g. mozilla.org, theverge.com). Without the header Focus sees and blocks the trackers and counts them. This makes it look like those websites try to track the user ("mozilla has trackers too!!") even though they honor the user's decision.

  2. In theory trackers will get de-listed from Disconnect when they start complying with DNT. This means after updating our lists we'd suddenly not block those trackers AND do not tell them that we do not want to be tracked. Firefox always enables DNT in private browsing mode with tracking protection.

Also note that:

  • We do not store any data. So traditional "cookie" tracking doesn't work across sessions regardless.

  • It seems to be impossible to add DNT headers to all requests WebView makes.

@pocmo pocmo changed the title Add DNT header Add DNT header (Do Not Track) Jul 12, 2017
@pocmo
Copy link
Contributor

pocmo commented Jul 18, 2017

Should be done when we update the block lists (#856)

@bbinto bbinto added this to the Reserve Backlog milestone Nov 10, 2017
@bbinto bbinto added the P3 label Nov 10, 2017
@bbinto
Copy link
Contributor

bbinto commented Nov 10, 2017

I wanted to resurface this and #856.

This means after updating our lists we'd suddenly not block those trackers AND do not tell them that we do not want to be tracked.

Not quite sure if I understand this correctly. Doesn't the DNT header DO tell them that we don't want to be tracked?

@pocmo
Copy link
Contributor

pocmo commented Nov 10, 2017

Not quite sure if I understand this correctly. Doesn't the DNT header DO tell them that we don't want to be tracked?

Yes, what I meant is: We need to send the DNT header in order to not get tracked by the sites that have been removed from the new lists. Right now we do not send the header so we would get tracked if a site is removed from the tracking protection lists.

@jenn-chaulk jenn-chaulk modified the milestone: Reserve Backlog Nov 10, 2017
@bbinto bbinto modified the milestones: Reserve Backlog, Focus Android V6.0 Nov 24, 2017
@bbinto bbinto modified the milestones: v6.0, v5.0 Feb 26, 2018
@pocmo
Copy link
Contributor

pocmo commented Feb 28, 2018

This is enabled for GeckoView-based builds.

@bbinto bbinto modified the milestones: v6.0 (GeckoView), v5.0 Mar 13, 2018
@bbinto bbinto modified the milestones: v6.0 (GeckoView), Backlog Apr 26, 2018
@bbinto bbinto added P4 and removed P3 labels Apr 26, 2018
@bbinto bbinto modified the milestones: Backlog (P2, P3), Backlog (>P3) Jun 4, 2018
@rpappalax rpappalax added testing meta label for all features/bugs requiring test eng:automation Build automation, Continuous integration, .. labels Jul 27, 2018
@Sdaswani
Copy link

@ekager can we close this issue? I see a closed PR and something related to GV?

@ekager
Copy link
Contributor

ekager commented Sep 14, 2018

Sounds like GV already has it automatically, let's close this

@ekager ekager closed this as completed Sep 14, 2018
@ghost ghost removed the feature label Sep 14, 2018
@bbinto
Copy link
Contributor

bbinto commented Sep 14, 2018

Do we expose this somehow via UI or is that auto-on? How would a user know? Anything we should highlight with comms as "comes automatically with gv" /cc ing @vesta0.

I couldn't find any ui in the reference issue

#2857 Add do not track header to settings > Privacy and Security

@ekager
Copy link
Contributor

ekager commented Sep 14, 2018

We get this for free in the switch to GV because we're always in private mode, it is auto-on.

You could include this in the GV marketing for the switch.

@vesta0 vesta0 removed the P4 label Sep 19, 2018
@vesta0 vesta0 modified the milestones: Backlog (>P3), Sprint 6 Sep 19, 2018
@vesta0 vesta0 added the feature label Oct 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:automation Build automation, Continuous integration, .. feature QAApproved testing meta label for all features/bugs requiring test
Projects
None yet
Development

No branches or pull requests