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

bug(YouTube/Hide ads): Hide view products banner setting not working #2437

Closed
6 tasks done
smed79 opened this issue Oct 17, 2024 · 14 comments
Closed
6 tasks done

bug(YouTube/Hide ads): Hide view products banner setting not working #2437

smed79 opened this issue Oct 17, 2024 · 14 comments
Labels
A/B test Reproducible only in some users Bug report Something isn't working

Comments

@smed79
Copy link

smed79 commented Oct 17, 2024

Type

  • Other (ads)

Tools used

  • RVX Manager v1.22.2
  • RVX Patchs v4.14.2

Application

  • YouTube v19.16.39

Bug description

View products banner that overly the video player is not hidden.

hvpd

bnr

Additional context

Example video when you can reproduce

  • https://youtu.be/tsdpdtmigXM
  • https://youtu.be/sM_lWr6iRds

Device Environment

  • Android 14.0 (OxygenOS)

RVX Logs

- Device Info
RVX Manager: 1.22.2
Model: CPH2581
Android version: 14 (OxygenOS)
Supported architectures: arm64-v8a, armeabi-v7a, armeabi
Root permissions: No

- Patch Info
App: rvx.android.youtube v19.16.39 (Suggested: Any)
Patches version: v4.14.2
Patches added: Custom header for YouTube, Force hide player buttons background, Hide Shorts dimming, Hide shortcuts, MaterialYou
Patches removed: None
Default patch options changed: None

- Settings
Allow changing patch selection: true
Version compatibility check: true
Show universal patches: false
Remove unused library: true
Use pre-release: false
Patches source: inotia00/revanced-patches
Integration source: inotia00/revanced-integrations

- Logs
Reading APK
Decoding app manifest
Loading patches
Merging integrations
Deleting existing temporary files directory
Decoding resources
Executing patches
Applied 61 patches
Compiling patched dex files
Compiled 8 dex files
Compiling modified resources
Ripping selected native libs
Aligning APK
Signing APK
Patched APK
Installing patched app
Installed

Acknowledgements

  • This issue does not reproduce on unpatched YouTube or YT Music.
  • This issue is not a duplicate of an existing bug report.
  • I didn't use any settings marked as Experimental Flags.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
  • I have written the title and contents in English.
@smed79 smed79 added the Bug report Something isn't working label Oct 17, 2024
@inotia00
Copy link
Owner

please add the version of revanced-patches you used

@inotia00 inotia00 added the Waiting on author OP's answer is needed label Oct 17, 2024
@smed79
Copy link
Author

smed79 commented Oct 17, 2024

More details add above ☝️

I get more big annoying banner on the below 👇 video

  • https://youtu.be/pijEZbZemUM

Screenshots

001

002

Thank you in advance 🙏

@inotia00 inotia00 removed the Waiting on author OP's answer is needed label Oct 17, 2024
inotia00 added a commit to inotia00/revanced-integrations that referenced this issue Oct 18, 2024
@smed79
Copy link
Author

smed79 commented Oct 19, 2024

Applying the last patches v4.15.1-dev.3 unfortunately did not fixed the issue.

What i can provide to help you investigate?


PS: I also noticed that "Up next" suggested video banner is not hidden with "Hide end screen card and suggested video" switched on (autoplay is off).

Should i open new issue for this bug or just follow tracking it here?

@inotia00
Copy link
Owner

PS: I also noticed that "Up next" suggested video banner is not hidden with "Hide end screen card and suggested video" switched on (autoplay is off).

This is autoplay preview container, not end screen card and suggested video

▼ autoplay preview container ▼

hide-autoplay-preview

@inotia00
Copy link
Owner

inotia00 commented Oct 19, 2024

What i can provide to help you investigate?

If it's not hidden even in the latest dev branch, then this is a new ad that is not being reproduced on my side.

You can extract the debug log by following the steps below:

  1. Install ADB on your PC according to the docs.
  2. Turn on Enable debug logging and Enable dubug buffer logging in RVX settings.
  3. Enable USB debugging in Developer options in System settings.
  4. Connect your phone and PC, and open CMD in the directory where adb.exe is.
  5. Type the following in CMD:
adb logcat | findstr "LithoFilterPatch" > log.txt
  1. Open a video with the view products banner.
  2. Type Ctrl + C in CMD.
  3. Attach the log.txt.

@inotia00 inotia00 added the A/B test Reproducible only in some users label Oct 19, 2024
@inotia00 inotia00 changed the title bug: View Products banned is not hidden (ads) bug(YouTube/Hide ads): Hide view products banner setting not working Oct 19, 2024
@smed79
Copy link
Author

smed79 commented Oct 19, 2024

You can extract the debug log by following the steps below

I'm on Linux, so I used grep instead of findstr but unfortunately i get an empty file.

logcat

Filtering the logcat by the package name

adb logcat | grep rvx.android.youtube > log.txt

give me the output > 64f628f5af0dd2e64339d34d0d99d580


This is autoplay preview container, not end screen card and suggested video

Thank you very much 🙏

@inotia00
Copy link
Owner

No debug logging included

Are these turned on?

2. Turn on Enable debug logging and Enable dubug buffer logging in RVX settings.

Command in Linux:

adb logcat | grep --line-buffered "LithoFilterPatch" > log.txt

@smed79
Copy link
Author

smed79 commented Oct 20, 2024

Of course yes, Enable debug logging and Enable dubug buffer logging in RVX settings are turned on and also USB debugging in Developer Options but i still get an empty log.txt file with the command you have provided for linux.

I also tried via Wireless debugging using Shizuku + aShell You > Same thing, i get an empty file.

I don't know, maybe something goes wrong with my rom/device.
I will try to find another device for testing and see if i can reproduce the ads, then providing a log.

Thank you so much for your assistance.

@inotia00
Copy link
Owner

You can also extract logs via the Locat Reader app

Try this:

  1. Download and install the Locat Reader app:

F-Droid (summary)
F-Droid (direct download)

  1. Enable USB debugging and grant permission by entering the following command from the CMD on your PC:
adb shell "pm grant com.dp.logcatapp android.permission.READ_LOGS && am force-stop com.dp.logcatapp"
  1. Launch the Locat Reader app

  2. Move to 3dot menu - Filters

  3. Click the + button and type it like a screenshot:

LogCatReader

and then, click OK

  1. In RVX, turn on the following settings:
Settings - ReVanced Extended - Miscellaneous - Enable debug logging / Enable dubug buffer logging
  1. Force close and launch the RVX again

  2. Open a video with the view products banner

  3. After about 10 seconds, back to the Locat Reader app and click 3dot menu - Save

3Dots

  1. Attach the log.txt

@smed79
Copy link
Author

smed79 commented Oct 20, 2024

Hope this help > logcat_10-20-2024_08-28-01.txt.zip

... & Thanks for the useful tutorial 👍

@inotia00
Copy link
Owner

Hope this help > logcat_10-20-2024_08-28-01.txt.zip

... & Thanks for the useful tutorial 👍

The debug log contains the keyword shopping_overlay.eml-js-b, which is missing from both ReVanced and RVX

Can you please add the keyword shopping_overlay to Settings - ReVanced Extended - General - Custom filter - Custom filter and report back if it gets hidden?

@smed79
Copy link
Author

smed79 commented Oct 20, 2024

I confirm... Yes.

The added filter: shopping_overlay, successfully hide the shopping products banner 🎉

@smed79
Copy link
Author

smed79 commented Oct 20, 2024

The patches v4.15.1-dev.4 successfully Hide products banner.

Closing...

@smed79 smed79 closed this as completed Oct 20, 2024
@smed79
Copy link
Author

smed79 commented Oct 21, 2024

@inotia00 FYI, I figured out that what was causing to get the empty log file is that Logger buffer sizes was turned off by mistake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A/B test Reproducible only in some users Bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants