wielkie sprzątanie - Githubissues

MajkiIT commented 7 years ago

@F4z @xxcriticxx @hawkeye116477 @blocker999 @gzenio22

Chłopaki reguł nie ubywa i nie chcę pchać w listę wszystkiego, dlatego pokusiłem się o napisacie czegoś w php aby wklejać listę a skrypty sparsują i wyciągną z tego domeny, wywalą duplikaty i sprawdzą czy domena istnieje. Jeśli nie to w wynikach napiszę, że nie istnieje. Zrobiłem już coś takiego dla listy KAD ==> https://github.com/azet12/KAD/issues/454 . Jak skończę to zaczniemy to robić z automatu.

azet12 commented 7 years ago

Napiszę do niego maila. Kiedyś była podobna sytuacją z tobą, @xxcriticxx - twój profil na trochę zniknął wraz z twoimi wszystkimi issues, pusto tu było :-).

xxcriticxx commented 7 years ago

tak mie bot zamkna za spamowanie trzeba bylo email napisac czemu troche zajelo ale konto zostalo ponownie otwarte

xxcriticxx commented 7 years ago

@hawkeye116477 pomoge tobie ale ja nie posiadam konta na zadnej social network zabardzo mnie to nie interesuje

hawkeye116477 commented 7 years ago

@xxcriticxx Ok. Tyle, że co to ma wspólnego z social network?

xxcriticxx commented 7 years ago

@hawkeye116477 nie wiem myslalem ze to jakies filtry do social byly

hawkeye116477 commented 7 years ago

@xxcriticxx Mi chodziło bardziej o PopupBlockera, czyli

Polski filtr typu annoyances blokujący okienka zachęcające do rejestracji lub z newsami, promocjami i rabatami. Blokuje on widżety które dezinformują i odwracają uwagę, próbując na siłe wcisnąć swój produkt.

Filtrami social to też faktycznie się zajmował, ale chyba jakoś rzadko, dlatego ja ostatnio wysyłałem parę pull requestów. Ale do filtrów social nie trzeba mieć konta na socialach, filtry social są do blokowania elementów social na zwykłych stronach.

xxcriticxx commented 7 years ago

@hawkeye116477 jak sie zajmiesz to pomoge pisz co mam robic

hawkeye116477 commented 7 years ago

@xxcriticxx Ok, szukaj stron na których są te irytujące elementy i zgłaszaj je w issues Popup Blockera. @azet12 Jakby blocker dostał banana na IP lub nie chciało mu już się, to poproszę o odpowiednie prawa do comittowania, postaram się go godnie zastąpić :smile:.

xxcriticxx commented 7 years ago

@hawkeye116477 zrobisz forka czy podajemu tutaj narazie?

hawkeye116477 commented 7 years ago

@xxcriticxx Podaj do forka na razie --> https://github.com/hawkeye116477/PopupBlocker. Choć myślę, że raczej niedługo zaczniemy pracować na repo azeta, jeśli on może przekazać uprawnienia. Tak chyba byłoby lepiej, bo nikt nie musiałby adresu zmieniać.

xxcriticxx commented 7 years ago

@hawkeye116477 czego szukamy newsletter?

hawkeye116477 commented 7 years ago

@xxcriticxx Newsletter i inne. Okienka zachęcające do rejestracji lub z newsami, promocjami i rabatami, widżety które dezinformują i odwracają uwagę, próbując na siłe wcisnąć swój produkt.

xxcriticxx commented 7 years ago

ok zamontowalem filtry i niedlugo zaczynamy

xxcriticxx commented 7 years ago

@hawkeye116477 a filtry social przerabiamy?

hawkeye116477 commented 7 years ago

@xxcriticxx Ta, jak coś znajdziesz socialowego to zgłaszaj tu, a ja zobaczę co da się z tym zrobić :smile:.

xxcriticxx commented 7 years ago

moment musze filtry zamontowac

hawkeye116477 commented 7 years ago

@xxcriticxx Ok

xxcriticxx commented 7 years ago

👍

xxcriticxx commented 7 years ago

mam teraz takie advance filtry ze od godziny nic nie moge znalesc

azet12 commented 7 years ago

@xxcriticxx Blocker doświadczył tego samego problemu, co ty. Pomogę mu z odzyskaniem konta. Nigdzie nie odszedł.

hawkeye116477 commented 7 years ago

@azet12 Czyli banana dostał za spamowanie?

azet12 commented 7 years ago

@hawkeye116477 GitHub ma automatyczne boty wykrywające lewe konta i z jakiegoś powodu zablokowali blockera (trochę ironiczne). Nie ma to nic wspólnego z tym dość wrażliwym gościem, o którym wspomniałeś.

hawkeye116477 commented 7 years ago

@azet12 To co on taki niegrzeczny, rozrabiaka :smiley:.

MajkiIT commented 7 years ago

@azet12 Dzięki za info.

@all w przyszłym tygodniu odpalę stronkę do badania dead pages z list.

xxcriticxx commented 7 years ago

@azet12 dzięki za update miło ze nas odwiedziłeś

hawkeye116477 commented 7 years ago

@MajkiIT A skoro o sprzątaniu mowa to mam pytanie czy są w tym repo jacyś zbanowani i czy ewentualnie można byłoby przekazać tę listę do @azet12 i @blocker999, żeby też wpisali tych niegrzecznych/trolli na listę?

xxcriticxx commented 7 years ago

@hawkeye116477 ???

hawkeye116477 commented 7 years ago

@xxcriticxx Chodzi mi o listę bardzo niegrzecznych ludzi/trolli, no wiesz nazir, itd. :smile:.

xxcriticxx commented 7 years ago

@hawkeye116477 sie steskniles za nim?

MajkiIT commented 7 years ago

@hawkeye116477 coś wyciągnę i wyślę na maila do nich. Tam były chyba 2-3 takich osób.

krystian3w commented 4 years ago

~~Chyba fail, on widzi uBO suplement i AG suplement = wciąż nie mamy pliku wspólnego.~~

~~więc narzędzie może domaga się tego: #16769 - pytanie jak stare uBO wtedy będziemy wspierać.~~

Już przerobili w większości tekst "REKLAMA" na ::before/::after z wciśniętymi znakami o zerowej szerokości (i widzialności pewnie też).

Dodatkowo niby rozdzielone są na 3 linie:

https://github.com/MajkiIT/polish-ads-filter/blob/9cc17dbf834447761623c885d53a4e9d54f54f42/polish-adblock-filters/adblock_ublock.txt#L148 https://github.com/MajkiIT/polish-ads-filter/blob/9cc17dbf834447761623c885d53a4e9d54f54f42/polish-adblock-filters/adblock_ublock.txt#L154 https://github.com/MajkiIT/polish-ads-filter/blob/9cc17dbf834447761623c885d53a4e9d54f54f42/polish-adblock-filters/adblock_ublock.txt#L158

czyli warto też odpalić tryb "FOP".

Niby dobrze by było też je w końcu przepisać na :not(:has(foo)):

##div[class]:has(>div[class]:first-child:has-text(REKLAMA):not(:has(>*)))

Jak dalej coś wycinają, a mogą tylko męczyć CPU na marne.

Opcjonalnie może nowsze upward():

##div[class]:first-child:has-text(REKLAMA):not(:has(>*)):upward(1)
##div[class]:first-child:has-text(REKLAMA):not(:has(>*)):upward(div) - ten może za dużo łapać, np.:

div > span > div"REKLAMA".

zaba-monika commented 4 years ago

najpierw przypomnijmy o pewnej osobie, która ciężko tu kiedyś pomagała i umieściła tu kilka wielkich komentarzy w których wyzgłaszała dużo: duplikatów, reguł zbędnych, reguł błędnych / losowych paździerzy przez które sypało reklamami itp., a tu jacyś $%%^^ z administracji github.com wszystko po chamsku pokasowali w kilka sekund, ech.. (nie mam tu na myśli admina tego repo, tylko adminów github.com).

█████████████████████████████████████████████████████████████████████████

tak czy inaczej umieszczam kolejny komentarz z błędnymi regułami:

Invalid network filter in POL-0: ||tkchopin.pl/cams/video_file/*$media,rewrite=abp-resource:blank-mp3,domain=tkchopin.pl
Invalid extended filter in POL-0: www.bolec.info#?#.bottom:nth-of-type(n+5):nth-of-type(-n+6) > .bbbb:-abp-contains(reklama), div.bottom:nth-of-type(21) > .bbbb > div
Invalid extended filter in POL-0: www.bankier.pl##li.m-home-link-list__item:has-text(/reklama/), .o-home-ecom-box
Invalid extended filter in POL-0: ticketmaster.pl#?#div[class^="sc-"]:-abp-contains(Reklama), [id^="ad-slot-"]
Invalid extended filter in POL-0: gazetabilgoraj.pl#?#.vc_row-has-fill.vc_row-fluid.wpb_row.vc_row.row:-abp-has(img[width="350"][height="250"][src*="-350x250."]), .vc_row-has-fill.vc_custom_1526373595932.vc_row-fluid.wpb_row.vc_row.row, .vc_row-has-fill.vc_custom_1526373595932.vc_row-fluid.wpb_row.vc_row.row
Invalid extended filter in POL-0: braclowiecka.pl#?##sidebar > div.widget > h3:-abp-contains(Reklama), div.widget > .socialize

nie wolno łączyć filtrów specjalnych (#?#) z innymi: ticketmaster.pl#?#div[class^="sc-"]:-abp-contains(Reklama), [id^="ad-slot-"] - takie reguły nie będą w ogóle działały
a w dzisiejszym filtrze nawet 3 błędy na raz: www.bankier.pl##li.m-home-link-list__item:has-text(/reklama/), .o-home-ecom-box a) pomijając, że filtr specjalny połączony z innym b) to dodatkowo powinno być raczej :-abp-contains zamiast :has-text i wtedy dodatkowo ? w ##
co do ,rewrite to chyba już to wycofali z ABP po wykrytej luce bezpieczeństwa, albo go przynajmniej ograniczyli

█████████████████████████████████████████████████████████████████████████

no i jeszcze pamiętać o "zdublowanej" REGULE:

█████████████████████████████████████████████████████████████████████████

warto wspomnieć w celach humorystycznych również o kilku błędach / literówkach oraz zawirowaniach dotyczących działania reguł, u krystian3w: a) https://github.com/uBlockOrigin/uBlock-issues/issues/1339#issuecomment-727552957 b) https://github.com/AdguardTeam/AdguardFilters/issues/67537#issuecomment-727549743

krystian3w commented 4 years ago

Invalid network filter in POL-0: ||tkchopin.pl/cams/video_file/*$media,rewrite=abp-resource:blank-mp3,domain=tkchopin.pl

https://help.eyeo.com/en/adblockplus/how-to-write-filters#rewrite

(najnowsze rewrite zmieniono w znane już redirect= z uBO i nowszego AdGuard).

Poprawna - działa z AdBlock / ABP, uBO ma swoją w suplemencie, AG chyba podobną w AdGuard Base.

krystian3w commented 4 years ago

@MajkiIT dopytaj może Wietnamczyków o ich kod strony:

https://abpvn.com/ruleChecker/redundantRuleChecker.html

https://github.com/abpvn/abp-rule-checker - tu niby mają jakiś z maja 2020.

hawkeye116477 commented 4 years ago

Wg listy zmian w maju nic nie zmienili, a commit https://github.com/abpvn/abp-rule-checker/commit/95f663568010914b58077f4d1c0dc8517f317244 mówi o jakimś rozszerzeniu. Jest tylko 1 zmiana z września 2018.

krystian3w commented 4 years ago

Niby dodali CSP:

https://github.com/abpvn/abp-rule-checker/commit/1004ade0108abf83b0bd0682e0a446ceabf51b3a

Webrtc / websocket:

https://github.com/abpvn/abp-rule-checker/commit/43b5ffe509876ebe4d1af6337cdc5efa86ea2ff5

Bo ich kod niby nie reaguje na rewrite=abp-resource:blank-mp3.

zaba-monika commented 4 years ago

1

https://github.com/MajkiIT/polish-ads-filter/issues/3318#issuecomment-727574408 : (najnowsze rewrite zmieniono w znane już redirect= z uBO i nowszego AdGuard).

nigdzie nie mogę się doszukać żeby gdzieś w ABP zmienili rewrite na redirect, jeśli mógłbyś zarzucić jakimś linkiem...
dokładnie tego nie badałam, ale tak pobieżnie to wychodzi na to, że tego typu reguły zdaje się nie są kompatybline pomiędzy różnymi adblockerami, tzn. inne adblockery nie konwertują jej na swoje odpowiedniki (chyba, że tylko pomiędzy uBO <==> AG), więc nie jest to błąd na liście, tylko po prostu pod uBO ta reguła nie zadziała: a) ABP ma rewrite b) uBO ma redirect c) AG ma redirect + mp4
więc jeśli już trzeba, to każdemu adblockerowi należy osobną-własną regułę (ewentualnie dla uBO i AG wystarczy jedna jeśli ich redirect jest wzajemnie kompatybilny), więc dlatego admin nie usunął jej w SWOIM PR

2

co do https://github.com/MajkiIT/polish-ads-filter/pull/17636 - to nie sądziłem, że aż tyle tam jeszcze błędów było na liście, dobrze, że zostały wyłapane, admina trzeba pilnować bo z czasem trochę sieczka się robi z listy hehe

krystian3w commented 4 years ago

Tu masz opis obecnego rewrite:

https://help.eyeo.com/en/adblockplus/how-to-write-filters#rewrite

Obecnie działa już tylko przekierowanie na zneutralizowane lokalne pliki "abp-resource:", czyli to jest redirect ze "złą" nazwą.

ABP w wersji przestarzałej (poniżej 3.5.2) ma zablokowaną instalację w Firefox, jako że dodatek niby może być "niestabilny"/"niebezpieczny" - czyli nawet nie można zrobić jakiegoś nowszego testu czy na mapach Google wywołasz alert() - czyli do testu musisz użyć wersji z pliku CRX/NEX (ZIP) w Chromium (a Chromowate nie trzymają starszych wersji na oficjalnych stronach sklepu z dodatkami) lub w Firefox tryb debugowania dodatku lub jak mu podmienisz ID (co raczej nie naraża 10% użytkowników Firefox, bo nie debugują dodatków i nie instalują ręcznie niepodpisanych).

zaba-monika commented 4 years ago

Acha, to w takim razie chyba powinieneś do nich napisać / poinformować ich o tym, żeby poprawili nazwę.

zaba-monika commented 3 years ago

Invalid extended filters in:

a) https://raw.githubusercontent.com/MajkiIT/polish-ads-filter/master/cookies_filters/adblock_cookies.txt: paczkomaty.pl##.modal:has(app-regulations-modal > .modal-header > .modal-title:has-text(RODO)), .modal-backdrop
b) https://gitcdn.xyz/repo/MajkiIT/polish-ads-filter/master/adblock_social_filters/adblock_social_list.txt: swietarodzina.pila.pl##^div.partial_staged_slideL.single_fading_slide:has-text(Skontaktuj się z nami korzystając z serwisów społecznościowych), div.slider_nav > a:nth-of-type(5)
c) psychowiedza.com#?#.vc_column-inner .cesis_icon_paragraph_ctn[id]:-abp-has( > a[href*="facebook.com"]

W błędach a) i b) to samo co w poprzednich błędach w https://github.com/MajkiIT/polish-ads-filter/issues/3318#issuecomment-727558349, czyli :

niedozwolone łączenie filtrów specjalnych z kolejnym filtrem po przecinku, trzeba rozdzielić filtry
dodatkowo powinno być raczej :-abp-contains zamiast :has-text i :-abp-has zamiast :has , wtedy dodatkowo ? w ## (przynajmniej w przypadku filtru a) bo w b) jest filtr html, który i tak działa tylko w uBO, ale dla ogólnej zasady też można zamienić na zgodny z ABP, albo może nie, sama nie wiem...zadecydujcie sami)

W błędzie c) brak okrągłego nawiasu ) na końcu

krystian3w commented 3 years ago

Gitlab nie gryzie, samodzielnie zaproponuj zmianę nazwy.

krystian3w commented 3 years ago

raczej :-abp-contains zamiast :has-text

komunikaty_suplement.txt blokowane przewijanie, jak je odblokowujesz w ABP (?), to możemy przenieść filtr.

github-actions[bot] commented 11 months ago

Ten wątek został automatycznie zablokowany, ponieważ 60 dni po jego zamknięciu nie było żadnej aktywności. Proszę otworzyć nowe zgłoszenie dla powiązanych problemów.

MajkiIT / polish-ads-filter

wielkie sprzątanie #3318

1

2