Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

LA Times - native ads #7454

Closed
lukemulks opened this issue Mar 2, 2017 · 4 comments
Closed

LA Times - native ads #7454

lukemulks opened this issue Mar 2, 2017 · 4 comments

Comments

@lukemulks
Copy link
Collaborator

lukemulks commented Mar 2, 2017

  • Did you search for similar issues before submitting this one?
    Y
  • Describe the issue you encountered:
    Reports of an ad with a nasty clickthrough on LATimes.com - not a content blocking overlay, but an actual ad. Appears similar to a sunday morning newspaper insert, which links to a "support us" page, and then finally lets user see the article content.

Also observed, native ads in the page, served by a new ad implementation not previously observed earlier this week.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Reported from MacOS, other platforms TBD.

  • Brave Version (revision SHA):
    0.13.4

  • Steps to reproduce:
    1.go to Latimes.com
    2.click article
    3.see ad

  • Actual result:
    Native ads.

  • Expected result:
    No native ads.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    TBD

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    TBD

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    These ads were confirmed on windows:
    brave-latimes-recommended-stories-03022017

brave-latimes-sponsoredpost-03022017

Have been unable to repro the "newspaper insert" ad from Windows, but the insert ad was reported from MacOS X, in London (could be a day-parting and/or regional issue)

Confirmed the following filter rules will resolve the two ad versions screencapped from windows, tested in about:adblock :

||aggrego.org^$script,image, domain=latimes.com
||adserve.postrelease.com^$script,image, domain=latimes.com
||troncdata.com^$script,image, domain=latimes.com
||polarmobile.com^$script,image, domain=latimes.com
||ntv.io^$script,image, domain=latimes.com
||gigya.com^$script,image, domain=latimes.com

Will file an issue and submit a PR in adblock-lists for the above. The fix above may resolve for the insert ad as well, but I need to repro from a mac to confirm.

@lukemulks lukemulks self-assigned this Mar 2, 2017
@lukemulks lukemulks added feature/adblock needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. site-bug labels Mar 2, 2017
@lukemulks lukemulks changed the title LA Times - Ad overlay with nasty clickthrough LA Times - native ads Mar 2, 2017
@lukemulks
Copy link
Collaborator Author

cc @BrendanEich ☝️

@lukemulks
Copy link
Collaborator Author

Issue filed in adblock-lists: brave/adblock-lists#8
PR to resolve the "sponsored content" ads and tracking: brave/adblock-lists#9

Will try and repro newspaper insert issue in MacOS.

@lukemulks
Copy link
Collaborator Author

Also having reports of intermittent adblocking overlays that match the past fix - seeing this called out on the uBlock forums and other anti-adblock lists that I'm tracking.

Going to check that and the native ad issue on macOS, as that appears to be the platform where this is most often reported/reproduced. I've not seen it after +50 page loads and 2 cache-clearings on Windows.

@lukemulks
Copy link
Collaborator Author

Confirming that after testing on macOS, the rules applied and submitted in adblock-lists here:

correct for these ad types:
https://cloud.githubusercontent.com/assets/8979240/23528497/ed0e94fc-ff4e-11e6-826c-dc30677a1e45.PNG

https://cloud.githubusercontent.com/assets/8979240/23528486/e82a19ac-ff4e-11e6-8b3e-9622e7ba5aa7.PNG

Unable to repro the adblock message that reported as being observed intermittently today, but will try a few things to force that to show up.

Received confirmation that the news-insert style ad that was observed was observed w/shields down.

Going to close this issue and resolve further from the brave/adblock-lists issue.

If the subscription prompt persists, I'll reopen this and dive deeper if I can't repro otherwise.

@cndouglas cndouglas removed the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Mar 31, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants