After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 796245 - Fanboy Annoyance list breaks 2 pages without ads
Fanboy Annoyance list breaks 2 pages without ads
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
3.28.x
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-05-18 19:06 UTC by Bjørn Lie
Modified: 2018-05-28 13:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Posten.no "broken" (94.95 KB, image/png)
2018-05-18 19:06 UTC, Bjørn Lie
Details
Posten.no ok (578.13 KB, image/png)
2018-05-18 19:07 UTC, Bjørn Lie
Details
Yr.no - twitter insert broken (256.63 KB, image/png)
2018-05-18 19:07 UTC, Bjørn Lie
Details
yr.no twitter insert ok (379.73 KB, image/png)
2018-05-18 19:08 UTC, Bjørn Lie
Details

Description Bjørn Lie 2018-05-18 19:06:38 UTC
Created attachment 372199 [details]
Posten.no "broken"

2 different pages gets "broken" when having Fanboy Annoyance list active in adblock.

posten.no
and
yr.no

Please see attached screenshots.

Removing Fanboy Annoyance list, but keeping the other 2 default block lists makes the pages render as intended.
Comment 1 Bjørn Lie 2018-05-18 19:07:06 UTC
Created attachment 372200 [details]
Posten.no ok
Comment 2 Bjørn Lie 2018-05-18 19:07:43 UTC
Created attachment 372201 [details]
Yr.no - twitter insert broken
Comment 3 Bjørn Lie 2018-05-18 19:08:06 UTC
Created attachment 372202 [details]
yr.no twitter insert ok
Comment 4 Michael Catanzaro 2018-05-18 19:40:21 UTC
(In reply to Bjørn Lie from comment #0)
> Removing Fanboy Annoyance list, but keeping the other 2 default block lists
> makes the pages render as intended.

It's also breaking a webpage on my bank's website. Should probably just remove this list.
Comment 5 Michael Catanzaro 2018-05-19 16:49:16 UTC
(In reply to Michael Catanzaro from comment #4)
> It's also breaking a webpage on my bank's website. Should probably just
> remove this list.

Also breaks https://bugs.webkit.org/show_bug.cgi?id=185504

So we're up to four broken webpages discovered in recent days. Who knows what else it's breaking. Time to go.
Comment 6 Michael Catanzaro 2018-05-22 02:55:39 UTC
(In reply to Michael Catanzaro from comment #5)
> Who knows what else it's breaking.

Picture slideshows on a local news site... I'm really amazed how many websites this fixes. Guess I need to be more cautious with adblock filters in the future.
Comment 7 Michael Catanzaro 2018-05-22 02:57:24 UTC
Fixed for new users. I need to write a migrator to fix this up for existing users.
Comment 8 Michael Catanzaro 2018-05-22 04:45:10 UTC
(In reply to Michael Catanzaro from comment #7)
> Fixed for new users. I need to write a migrator to fix this up for existing
> users.

Done.

Dunno why git-bz decided not to post the patches here. They're in git....
Comment 9 Ryan Farmer 2018-05-28 02:14:31 UTC
This list was somehow breaking Facebook 1st party share buttons, which it never does in other browsers. 

I think I mentioned this in another bug report. 

That means something about Web's ad blocker is interpreting adblock plus rules differently than other browsers and this list was simply more destructive when loaded into Web than breaking parts of Facebook. :/
Comment 10 Michael Catanzaro 2018-05-28 13:43:52 UTC
Not surprising. Like I said on IRC, most problems with adblock are Epiphany's fault, we've never actually had a problem with a list before this bug. E.g. whitelisting is not implemented, so if the share buttons are blacklisted except whitelisted on facebook.com, that would not work. (But that can't be the cause of most of the problems on these sites: something else must have been wrong.)

The proper solution is to move to WebKit's content blocking, but that requires more work (starting with exposing WebKit API to make it usable).