Fix #239168 esladvice.com - #239592
Conversation
|
I wasn’t sure where to put this. It might be more accurate to put it in Tweaks, but I’m not sure, so I’ll leave it in the base for now. Someone can let me know if it should be moved. |
|
can you check on mobile device? |
On mobile, I get an anti adblock message regardless of the suggested rule. |
|
I’m not really sure why either, since the anti ad block overlay element is the same as on desktop, so it should already be blocked. |
|
It could be caused by filter optimization, but I would have to look into it further, as I’m not sure at the moment. |
|
on mobile device i see antiadblock + nuisance you described in your report even with filter in this PR below filters fix for me esladvice.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', 'mustClickAd0', 'prevent')
esladvice.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', 'mustClickAd1', 'prevent')
esladvice.com###delulu-overlay |
|
The anti-adblock issue is already fixed. It appears that we just need to exclude the rule from optimization. https://github.com/AdguardTeam/AdguardFilters/blob/a514e8945acafe51a09d9fe69847222f6bbf54fc/BaseFilter/sections/antiadblock.txt#L4390 however it is missing in https://filters.adtidy.org/android/filters/2_optimized.txt |
|
ok can you make necessary changes & make fix workable in all possible products? |
Example from |
can tell me link , STR ? i don't get redirected to biosjourney.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', 'viewTask0', 'prevent')
biosjourney.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', 'viewTask1', 'prevent') |
That is correct, and it works fine. I was just saying that it would get annoying to create a separate rule for every site, which is why I was trying to create one rule that would work on all of them. |
|
ok finally i got redirected(after constant reloads) to trash site. esladvice.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', '/mustClickAd\d+/', 'prevent')
biosjourney.com#%#//scriptlet('trusted-suppress-native-method', 'Storage.prototype.setItem', '/viewTask\d+/', 'prevent')
biosjourney.com,esladvice.com###delulu-overlay
biosjourney.com##.ad-728x250-wrapper |
|
All right, I updated it. Let me know if everything works all right for you. |
Creating the pull request
Prerequisites
To avoid invalid pull requests, please check and confirm following terms
What problem does the pull request fix?
If the problem does not fall under any category that is listed here, please write a comment below in corresponding section
What issue is being fixed?
Enter the issue address
Fix #239168
Terms