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

feat(YouTube): Add Check watch history domain name resolution patch #3537

Merged
merged 11 commits into from
Aug 15, 2024

Conversation

LisoUseInAIKyrios
Copy link
Contributor

@LisoUseInAIKyrios LisoUseInAIKyrios commented Aug 13, 2024

Detects if s.youtube.com is blocked by a third party ad blocker and displays a dialog informing the user.

This is caused by a problematic ad blocker list that some third party DNS ad blockers use without consideration.

Integration changes

warning

@cyberboh
Copy link

Will this work on DNS adblocker in router level and private DNS that using s.youtube.com in their adblocker list (e.g tiar.app?

@oSumAtrIX
Copy link
Member

As long as this matches, yes https://github.com/ReVanced/revanced-integrations/pull/675/files#diff-e63e83d793553aa220320047d9b2a55c55459ab63c9a4cdbcd9687e854e99baaR29-R33

Copy link
Member

@oSumAtrIX oSumAtrIX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A past patch was able to modify s.youtube.com to www.youtube.com. Perhaps it makes sense to fallback to www in case a blocker is detected. This would spare the user from yet another popup and can be triggered automatically without calling the user for action.

LisoUseInAIKyrios and others added 3 commits August 14, 2024 12:41
…DnsResolverVerificationPatch.kt

Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>
Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>
Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>
@LisoUseInAIKyrios
Copy link
Contributor Author

Automatically modifying s.youtube.com to www.youtube.com is a fix that might fail to work in the future, and then it's right back to the same problem except even more complicated.

This issue only shows up for users who installed a DNS blocker that uses that one problematic block list (other blockers such as AdGuard do not suffer this). So the user will still have issues with stock YT, using a browser, and everywhere else even if ReVanced tries to work around.

Instead of presenting this as "Hey your bad DNS blocker is incorrectly blocking something important that is not an ad, but we will try to fix it (no guarantees this will work)", it's simpler to just say "Your bad DNS blocker is misbehaving, here's how you fix it with 100% certainty".

@oSumAtrIX oSumAtrIX changed the title feat(YouTube): Detect DNS ad blockers that interfere with watch history feat(YouTube): Add Check watch history domain name resolution patch Aug 15, 2024
@LisoUseInAIKyrios LisoUseInAIKyrios merged commit 2af1425 into ReVanced:dev Aug 15, 2024
2 checks passed
@LisoUseInAIKyrios LisoUseInAIKyrios deleted the feat/detect_bad_dns branch August 15, 2024 16:06
revanced-bot pushed a commit that referenced this pull request Aug 15, 2024
# [4.13.0-dev.2](v4.13.0-dev.1...v4.13.0-dev.2) (2024-08-15)

### Features

* **YouTube:** Add `Check watch history domain name resolution` patch ([#3537](#3537)) ([2af1425](2af1425))
revanced-bot pushed a commit that referenced this pull request Aug 15, 2024
# [4.13.0](v4.12.0...v4.13.0) (2024-08-15)

### Bug Fixes

* **YouTube - GmsCore support:** Fix notifications not working by using the correct permissions ([19ddae2](19ddae2))

### Features

* **Google Photos:** Add `Spoof features` patch ([#3459](#3459)) ([7c218cd](7c218cd))
* **SCB Easy:** Remove broken `Remove debugging detection` patch ([#3518](#3518)) ([f4e23cb](f4e23cb))
* **YouTube:** Add `Check watch history domain name resolution` patch ([#3537](#3537)) ([2af1425](2af1425))
@martixy
Copy link

martixy commented Aug 18, 2024

I wanted to say thank you so much for this patch!

I had given up on watch history...

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

Successfully merging this pull request may close these issues.

5 participants