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.

Sign up for GitHub

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

Jump to bottom

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

Open
krystian3w opened this issue Jul 5, 2020 · 4 comments
Open

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

krystian3w opened this issue Jul 5, 2020 · 4 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@krystian3w
Copy link
Contributor 

@@/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
Copy link
Collaborator

hawkeye116477 commented Jul 5, 2020 •
edited
Loading

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
Copy link
Contributor Author

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

@krystian3w krystian3w changed the title Skrypt nie potrafi łączyć sieciowych dopuszczeń "@@||" / "@@/" Skrypt nie potrafi łączyć na czas sieciowych dopuszczeń "@@||" / "@@/" Jul 5, 2020
@hawkeye116477
Copy link
Collaborator

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 hawkeye116477 added the bug Something isn't working label Jul 5, 2020
@krystian3w krystian3w changed the title Skrypt nie potrafi łączyć na czas sieciowych dopuszczeń "@@||" / "@@/" Skrypt nie potrafi łączyć sieciowych dopuszczeń "@@||" / "@@/" wymieszanych z kosmetycznymi "#@#" Jul 5, 2020
@krystian3w krystian3w changed the title Skrypt nie potrafi łączyć sieciowych dopuszczeń "@@||" / "@@/" wymieszanych z kosmetycznymi "#@#" Skrypt nie potrafi czasem łączyć sieciowych dopuszczeń "@@||" / "@@/" wymieszanych z kosmetycznymi "#@#" Jul 5, 2020
@hawkeye116477
Copy link
Collaborator

hawkeye116477 commented Jul 5, 2020 •
edited by krystian3w
Loading

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

ScriptsPlayground/scripts/FOP.py

Line 199 in 60156c8

# last filter or filter didn't match regex or no domains

@hawkeye116477 hawkeye116477 added the help wanted Extra attention is needed label Aug 8, 2020
@krystian3w krystian3w mentioned this issue Oct 28, 2020
Closed
@krystian3w krystian3w removed their assignment Dec 24, 2020
@krystian3w krystian3w changed the title Skrypt nie potrafi czasem łączyć sieciowych dopuszczeń "@@||" / "@@/" wymieszanych z kosmetycznymi "#@#" The script sometimes fails to link network exceptions (@@) mixed with cosmetic exceptions (#@#) = bug in origin FOP logic Feb 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hawkeye116477 @krystian3w