search

FiltersHeroes / ScriptsPlayground

A place for testing and developing our scripts.
10 stars 4 forks source link

The script sometimes fails to link network exceptions (`@@`) mixed with cosmetic exceptions (`#@#`) = bug in origin FOP logic #5

Open krystian3w opened 4 years ago

krystian3w commented 4 years ago 
@@/plugins/uk-cookie-consent/assets/js/uk-cookie-consent-js.js$domain=3dwpraktyce.pl|...|xboxarcade.pl
@@/plugins/uk-cookie-consent/assets/js/uk-cookie-consent-js.js$domain=gieldykryptowalut.pl
@@||apis.google.com/js/platform.js$domain=lemonmind.com|olx.pl
@@||apis.google.com/js/platform.js$domain=naruto.oasgames.com

Na moje gremlinów nie widać.

hawkeye116477 commented 4 years ago

Sprawdziłem i jest ok, łączy.... To, że czasem zastrajkuje, to wcale nie musi oznaczać to co w tytule, problem może być inny.

krystian3w commented 4 years ago

W PR zduplikował, a socialowy to ma nawet #3 długą historię (kilka dni to raczej za długo).

https://github.com/MajkiIT/polish-ads-filter/pull/16779/files#diff-019c4d6c3918d69cd67e76aeec0a04a5R3313

obraz

https://github.com/MajkiIT/polish-ads-filter/pull/16743/files#diff-acd94f710b1c64f468452b4bff0e6a1bR6076

obraz
hawkeye116477 commented 4 years ago

Wychodzi na to, że problem jedynie jest wtedy jeśli reguły sieciowe i kosmetyczne są w jednym pliku, wtedy strzela focha i przestaje część łączyć.

hawkeye116477 commented 4 years ago

W zasadzie chyba niestety tak to zostało zaprojektowane, że albo on sortuje sieciowe albo kosmetyczne, razem nie.

https://github.com/FiltersHeroes/ScriptsPlayground/blob/60156c8137a5accea63638e1c01e4a124a807ac4/scripts/FOP.py#L199