-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bypassing classification by redirect through new tab or popup #50
Comments
Thanks for calling this out! I forgot to add a note to the spec, but we are planning to add handling for links in new windows, new tabs, and popups in an upcoming PR. I like your suggestion of copying the bounce tracking record to the new window - that's simple enough and preserves the correct |
FWIW, I think we've been mostly focused on same-tab redirections so far. We plan to work on popup flows in the future. I would advocate for leaving this as a TODO in the spec for now until we've had time to evaluate what use cases and sites this would impact. |
Sorry, I misunderstood earlier - our immediate plan is to add handling for extended navigations starting from a new tab, window, etc., or another opaque origin. Agree with Ben that the case of extended navigations through a new tab still needs further exploration. |
For the case where a popup opens on a tracking site, we can set the |
Looking at the spec wouldn't it be possible to avoid bounce tracker classification by performing redirects through a popup or a new tab (e.g.
target=_blank
)?Consider the following redirect chain: A.example -> Tracker.example -> B.example (shown below)
Since bounce tracking records are kept per-tab opening a popup would lead to the creation of a new record (for Window 2) with
initialHost = "Tracker.example"
. On extended navigation end, when running the record stateful bounces algorithm Tracker.example would then be excluded since it's set asinitialHost
.To prevent this bypass we could copy the bounce tracking record to the new tab / window.
The text was updated successfully, but these errors were encountered: