AdguardTeam / AdguardTeam/FiltersCompiler

Transform rules for Samsung Browser compatibility

Aperta
#2 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Priority: P3
Lingua principale
JavaScript
Stelle
64
Fork
11
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

@ameshkov commented on [Mon Aug 21 2017](https://github.com/AdguardTeam/AdguardFilters/issues/6385)

The problem is that Samsung cannot handle `^` symbol when it means the end of URL:

For instance:
`||google-analytics.com/analytics.js^` won't match `https://google-analytics.com/analytics.js`

To fix this issue we should add a duplicate rule with `|` instead of `^` for every rule matching this regexp:
`^(@@)?\|\|.+?/.+?\^(\$.*)?$`

For instance, this rule:
```
`||google-analytics.com/analytics.js^$third-party`
```

Should be replaced with two:
```
`||google-analytics.com/analytics.js^$third-party`
`||google-analytics.com/analytics.js|$third-party`
```

**It should be done for Content Blocker filters only**

---

@Mizzick commented on [Wed Oct 18 2017](https://github.com/AdguardTeam/AdguardFilters/issues/6385#issuecomment-337695856)

>It should be done for Content Blocker filters only

But what is Content FIlter? How to define it?

---

@ameshkov commented on [Sun Oct 22 2017](https://github.com/AdguardTeam/AdguardFilters/issues/6385#issuecomment-338481779)

> But what is Content FIlter? How to define it?

I mean rules for iOS, Safari, and for Android Content Blocker

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.