Skip to content
This repository has been archived by the owner on Mar 19, 2020. It is now read-only.

Issue 2980: Creating an initial list of storage trackers for STP #26

Merged
merged 1 commit into from
Jan 24, 2019

Conversation

jumde
Copy link
Contributor

@jumde jumde commented Jan 17, 2019

Fix brave/brave-browser#2980

Description

Smart tracking protection (or STP) will help block 1p storage trackers that are used to profile users (see: https://webkit.org/blog/8311/intelligent-tracking-prevention-2-0/) . This feature is in PR for brave-core brave/brave-core#403. StorageTrackingProtection.dat will be used by tracking_protection component to check if the redirection chain (on click the user goes through a list of intermediate domains before landing on the final domain) has any trackers.

If the user has tracking protection enabled for a domain, STP will check if the urls in the redirect chain are storage trackers and block access to the Storage APIs (localStorage/sessionStorage/IndexedDB/WebSQL) for the storage trackers.

Initial list is populated from the blog post: https://brave.com/redirection-based-tracking/

Going forward this list will be generated by the crawler that @snyderp worked on.

auditors: @bsclifton, @snyderp

@jumde jumde self-assigned this Jan 17, 2019
@jumde jumde requested review from bsclifton and SergeyZhukovsky and removed request for SergeyZhukovsky January 17, 2019 04:04
Copy link
Member

@SergeyZhukovsky SergeyZhukovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is that list supposed to be used?

@jumde
Copy link
Contributor Author

jumde commented Jan 19, 2019

How is that list supposed to be used?

@SergeyZhukovsky - Let me know if the description answers your question. I can add more details if needed.

@SergeyZhukovsky
Copy link
Member

How will that dat file apply to our existing list? Will it be a separate list or concatenated list?
Otherwise looks good to me.

@jumde
Copy link
Contributor Author

jumde commented Jan 19, 2019

@SergeyZhukovsky - This will be a separate list, the feature is independent of the existing tracking protection feature but will be part of the same component as the intent (blocking storage trackers) is related. The feature will be gated by runtime and build-time flags for the first iteration.

@jumde jumde mentioned this pull request Jan 19, 2019
17 tasks
Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

Copy link
Member

@SergeyZhukovsky SergeyZhukovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

@jumde jumde merged commit 132ac93 into master Jan 24, 2019
@bsclifton bsclifton deleted the storage_tracking branch January 24, 2019 18:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants