-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add AdGuard-trusted scriptlets to replace the old JS rules #106
Comments
https://github.com/AdguardTeam/AdguardFilters/blob/70ee34ef97c141408bbe45f646a7f4354cd3e8cb/AnnoyancesFilter/sections/cookies_specific.txt#L4704-L5073 - daily file have new lines above 4707 |
I'll keep a checklist with categorized ideas in the top comment. But I'll need examples for every kind of it. |
I suggest to mark new AdGuard-trusted scriptlets with a prefix |
Let's go with |
Regarding if ( document.getElementsByClassName("mghead").length < 1 ) {
console.log("adblock detected");
} so creating element with class |
Can you please add sessionStorage along with it? #118 |
Adblock Plus has a snippet |
@slavaleleka this one is interesting, but it should definitely be limited to trusted lists only. |
ABP still support all snippets form only anti-cv file or from own addons settings in place "my added rules". So no need fastly write system for privileges (can wait as mark use other snippet as safe loaded form any filter list file). |
We're adding support for "trusted" scriptlets to CoreLibs: AdguardTeam/CoreLibs#1612 Once it's added, we should start working on the new scriptlets. |
I suppose all world wait for official syntax. |
…-100 to master * commit '4bf88bcbcbe0e165e3c56fd35a687348a6c0a439': upd builds upd builds AG-100: typescript support
Regarding For example, here - https://hdfilmifullizle.com/13-mezar-izle |
These were implemented in 1.7.3 release:
The rest will be added in 1.8 |
With this we can replace script rules for AdguardTeam/AdguardFilters#132525, AdguardTeam/AdguardFilters#132814, AdguardTeam/AdguardFilters#135699 |
https://github.com/AdguardTeam/AdguardFilters/blob/master/AnnoyancesFilter/sections/cookies_specific.txt#L4707
There are tons of JS rules in the Annoyances filter that deal with cookie consent screens.
We should find a way to use scriptlets for that. We may need to introduce a "trusted" type of scriptlets for that, allowed to the user filter and our filter lists only, i.e.
trustLevel: full
.click()
an element: over 180 rules using this Add new scriptlet — for click action #23localStorage.setItem
: over 200 rules usinglocalStorage.setItem
document.cookie
.trusted-replace-xhr-response
#202The text was updated successfully, but these errors were encountered: