Skip to content
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

On firefox, clicking links in tweets shows a false positive blocked t.co domain on the resulting page #2692

Closed
simonft opened this issue Sep 26, 2020 · 3 comments · Fixed by #2896
Labels
bug Firefox heuristic Badger's core learning-what-to-block functionality

Comments

@simonft
Copy link

simonft commented Sep 26, 2020

When clicking a link on twitter, the link opens a t.co url in a new tab which redirects to the actual page. When using Privacy Badger on Firefox, a blocked t.co request will often be shown on the final page. E.g.:

image

This doesn't seem to happen with every link, thought it will pretty consistently happen for me with the links in these tweets:

https://twitter.com/PaulMitchum/status/1309920426152189952
https://twitter.com/varlogsimon/status/1309972367041585155

If I open the link with a middle click instead of a left click, the domain will only show up as blocked if I can click on the new tab before the redirect away from the t.co domain happens. Upon a refresh of the page, t.co no longer shows in the list of detected trackers.

I can't reproduce this in chrome.

I have no idea how any of this works under the hood but I'm guessing there's a race condition somewhere?

@bheerssen
Copy link

This happened to me also.
PB-bug

@ghostwords
Copy link
Member

Hello and thanks for reporting this bug!

It looks like we sometimes see a request for https://t.co/favicon.ico after the request for the main document of the destination page. This makes Privacy Badger think t.co is present on the destination page.

This is related to #1997 and should be fixed when we come back to finishing #2198.

@ghostwords
Copy link
Member

ghostwords commented Sep 28, 2020

My guess is favicon fetching in Firefox has to do with how long it takes for t.co to redirect. Chrome doesn't ever seem to fetch the favicon in this situation. Regardless, Firefox extension APIs at this point provide enough information for us to recognize that the t.co favicon request originated from a t.co "page", and as such (A) doesn't belong to the current page and (B) is first-party and should be left alone.

@simonft simonft changed the title On firefox, ciicking links in tweets shows a false positive blocked t.co domain on the resulting page On firefox, clicking links in tweets shows a false positive blocked t.co domain on the resulting page Sep 28, 2020
@ghostwords ghostwords added the heuristic Badger's core learning-what-to-block functionality label Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Firefox heuristic Badger's core learning-what-to-block functionality
Projects
None yet
3 participants