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

Brave shields are not working properly #14471

Closed
JohnnyX071 opened this issue Mar 4, 2021 · 18 comments · Fixed by brave/brave-core#8152
Closed

Brave shields are not working properly #14471

JohnnyX071 opened this issue Mar 4, 2021 · 18 comments · Fixed by brave/brave-core#8152

Comments

@JohnnyX071
Copy link

Since the last brave update, YouTube ads started showing up even though shields are on. I even tried putting them on aggressive but still. There are no steps to reproduce, it always happens. You can see some screenshots
Screenshot_6
Screenshot_7

I am using brave Version 1.21.73

@FoxMcloud5655
Copy link

I have this same issue. Likely, Google found a way to bypass it and Brave will have to find another way of blocking it. Unfortunately an endless cycle...

I tried blocking the entire element directly and it works until the page is refreshed.

@alekny
Copy link

alekny commented Mar 5, 2021

Same for me, as well as custom filters stopped working completely. You can add objects via rightclick->block and after a page refresh they are all back.

@ff2000
Copy link

ff2000 commented Mar 5, 2021

Same issue here.
Firefox with uBlock origin blocks the ads with no issue.

@ryanbr
Copy link

ryanbr commented Mar 5, 2021

The above screenshot is just cosmetic issue, see #11955 @JohnnyX071

Unless it's another issue with youtube?

@JohnnyX071
Copy link
Author

The above screenshot is just cosmetic issue, see #11955 @JohnnyX071

Unless it's another issue with youtube?

It's not just that, I get ads on some videos too but I don't have screenshot for that

@ff2000
Copy link

ff2000 commented Mar 5, 2021

Ad in a video:
Screenshot_20210305_135710
And when the video starts an ad pops on top of the video list:
Screenshot_20210305_135740
I have set aggressive blocking.

@tnup
Copy link

tnup commented Mar 5, 2021

same as @ff2000 said.

Version 1.21.73 Chromium: 89.0.4389.72 (Official Build) (64-bit)

@stephendonner
Copy link

stephendonner commented Mar 5, 2021

A good repro case to see the Skip Ads overlay— at least on nightly (and 1.21.73) — is to:

  1. sign in with a non-Premium account
  2. load a video
  3. click a video on the right from the suggested list
  4. click back
  5. now click forward

@LaurenWags
Copy link
Member

Able to reproduce issue from #14471 (comment) a few times using the below steps, but it was not consistent:

  1. Clean profile
  2. Navigate to youtube.com
  3. Login
  4. View a couple of videos (either in the same tab or open video in a new tab)
  5. Go back to youtube.com
  6. Right click on tab and select "Duplicate"
  7. Do step 6 a few times, one of the times you will get an "Ad" video in the first slot, other times it is blank

Screen Shot 2021-03-05 at 10 58 30

Brave | 1.21.73 Chromium: 89.0.4389.72 (Official Build) (x86_64)
-- | --
Revision | 3f345f156bfd157bd1bea06310e55f3fb2490359-refs/branch-heads/4389@{#1393}
OS | macOS Version 10.15.7 (Build 19H512)

@FoxMcloud5655
Copy link

That's because YouTube doesn't show an ad every single time, but yes, this is reproducible; can confirm.

@stephendonner
Copy link

With both nightly and 1.21.73: I should also mention another use-case, where, instead of ads, I see a blank/empty region (where others are seeing the ad, as already mentioned).

  1. log in to non-Premium account
  2. load youtube.com

I see a placeholder on the top-left spot, instead:
Screen Shot 2021-03-05 at 8 57 41 AM

@stephendonner
Copy link

There's also ads on nightly in the search-results page, where I don't see them using 1.19.88:

  1. log in to non-Premium account on `youtube.com
  2. search for, e.g. a car (lexus is 350): https://www.youtube.com/results?search_query=lexus+is+350
  3. look at the results page
1.19.98 nightly
Screen Shot 2021-03-05 at 9 46 02 AM Screen Shot 2021-03-05 at 9 46 07 AM

@LaurenWags
Copy link
Member

LaurenWags commented Mar 7, 2021

Verified passed with

Brave	1.21.74 Chromium: 89.0.4389.72 (Official Build) (64-bit)
Revision	3f345f156bfd157bd1bea06310e55f3fb2490359-refs/branch-heads/4389@{#1393}
OS	Linux

Reproduced issue originally reported under #14471 (comment) with 1.21.73. Was unable to reproduce using 1.21.74 after fix was applied using steps from #14471 (comment).

1.21.73 1.21.74
1 21 73 corner ad 1 21 74 corner ad

Verified PASSED with

Brave 1.21.74 Chromium: 89.0.4389.72 (Official Build) (x86_64)
Revision 3f345f156bfd157bd1bea06310e55f3fb2490359-refs/branch-heads/4389@{#1393}
OS macOS Version 11.2.2 (Build 20D80)

Used the steps to reproduce in #14471 (comment) and was able to see the issue using 1.21.73; updating to 1.21.74, I was no longer able to reproduce the original issue either.

1.21.73 1.21.74
Screen Shot 2021-03-06 at 6 48 01 PM Screen Shot 2021-03-06 at 6 46 23 PM

@beatschubser
Copy link

spiegel.de was full of ads with 1.21.73. Looks good again on 1.21.74. Thank you!

@ff2000
Copy link

ff2000 commented Mar 17, 2021

Just updated to 1.21.77 and ads are back.

@kjozwiak
Copy link
Member

@ff2000 which website where you seeing ads on? Would it be possible for you to add a screenshot? There's a few issues that we're aware of regarding a race condition in the cosmetic filter where sometimes ads will get through due to the script loading either too late or too early. Refreshing usually removes them. We're close to having a fix which will be deployed in Nightly first.

However, if you're seeing something different and the ads appearing all the time, let us know the website and we'll take a look/address it 👍

@ScamCast
Copy link

This is somewhat still an issue. The ad is blocked but leaves an empty space because of the parent "ytd-rich-item-renderer" element styling. That element would need to be hidden if it contains the "ytd-display-ad-renderer" element.

image
image
image

@ff2000
Copy link

ff2000 commented Mar 24, 2021

@ff2000 which website where you seeing ads on? Would it be possible for you to add a screenshot? There's a few issues that we're aware of regarding a race condition in the cosmetic filter where sometimes ads will get through due to the script loading either too late or too early. Refreshing usually removes them. We're close to having a fix which will be deployed in Nightly first.

However, if you're seeing something different and the ads appearing all the time, let us know the website and we'll take a look/address it

Today came another update. After restarting I got ads again. I didn't respond earlier because I wanted to wait for this update to verify my suspicion.

  • I closed brave with youtube being the active tab.
  • I updated brave
  • I restarted brave
  • The youtube tab loads and immediately shows an ad overlay - sometimes with an ad running, sometimes just a plain white overlay.
  • For some time I get in-video ads and the ad in the top left box on the youtube home page.
  • At some point ad blocking works again. I can't say ATM what exactly fixed the ad blocker.

So my issue might be unrelated but connected to updating brave.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment