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

Focus ad blocker breaks loading AMP pages on Google Search #6163

Closed
cramforce opened this issue Nov 12, 2016 · 11 comments
Closed

Focus ad blocker breaks loading AMP pages on Google Search #6163

cramforce opened this issue Nov 12, 2016 · 11 comments
Assignees

Comments

@cramforce
Copy link
Member

I received reports about this from users (See https://twitter.com/dwm189/status/797460628344541184 but it might affects other users)

This may, in particular, affect reddit pages, but maybe all pages.

To @choumx for debugging.

@cramforce
Copy link
Member Author

@cramforce
Copy link
Member Author

@dreamofabear
Copy link

  • Most news publishers unaffected
  • All reddit AMP pages broken by Block social trackers setting
  • All Forbes AMP pages broken by Block ad trackers setting
  • Block other content trackers setting has no apparent effect, despite scary warning

@cramforce
Copy link
Member Author

@dreamofabear
Copy link

dreamofabear commented Nov 14, 2016

I believe the problem is the way Focus builds the URL filtering regex from disconnect's blacklist JSON. Specifically, it creates Swift regexes for blacklisted domains that incorrectly match AMP CDN URLs. I'll add details on the Bugzilla entry that Malte filed above.

@dreamofabear
Copy link

PR with my proposed fix: mozilla-mobile/focus-ios#26

@dreamofabear
Copy link

/cc @dmccown

@dreamofabear
Copy link

PR is merged. Bug should be fixed on the next release of Focus.

@rudygalfi
Copy link
Contributor

This might be happening again based on reports on Reddit. @choumx Do you have any status on when the fix went out (if it went out) and can you check again to see if you can reproduce?

@dreamofabear
Copy link

Just tried and can't repro. The fix was released on November 17, 2016. That user may need to update the Firefox Focus app to v2.0+.

@cramforce
Copy link
Member Author

@rudygalfi Apparently it isn't entirely trivial to get focus to update the ruleset. I do not have instructions, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants