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.

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

hrkgame.com #18147

Closed
Alex-302 opened this issue Dec 10, 2020 · 11 comments
Closed

hrkgame.com #18147

Alex-302 opened this issue Dec 10, 2020 · 11 comments

Comments

@Alex-302
Copy link

Alex-302 commented Dec 10, 2020

Issue URL (Incorrect Blocking)

https://www.hrkgame.com/en/account/settings/security

Comment

Click on 2-Step Verification Activate > when adguard works - popup overlay will not show. When adguard is disabled - all is fine - popup works. You have to login to see this page.

AdguardTeam/AdguardFilters#69498
Caused by these rules from Polskie Filtry Ciasteczkowe:

7ryb.pl,etransport.pl,grupawsparcia.pl,hrkgame.com#$#.dimmable, .dimmed { overflow: auto !important; }
7ryb.pl,etransport.pl,grupawsparcia.pl,hrkgame.com##.modals.dimmer

I hope this is a correct repository)

Screenshots

Screenshot 1

Screenshot 1

Screenshot 2

Screenshot 2

System configuration

Information value
AdGuard product: AdGuard Browser extension v3.5.23
Browser: Chrome
Filters: Ad Blocking:
AdGuard Base,
Peter Lowe's Blocklist

Privacy:
AdGuard Tracking Protection

Social Widgets:
AdGuard Social Media

Annoyances:
AdGuard Annoyances,
Adblock Warning Removal List,
I don't care about cookies

Security:
Online Malicious URL Blocklist

Language-specific:
Official Polish filters for AdBlock, uBlock Origin & AdGuard,
Polskie Filtry Ciasteczkowe,
KAD - Przekręty,
Polish Anti Adblock Filters,
AlleBlock
@krystian3w
Copy link
Contributor

krystian3w commented Dec 10, 2020

@zaba-monika / @gwarser czemu tam nie chce działać watch-attr?

hrkgame.com#@#.modals.dimmer
hrkgame.com##.modals.dimmer.ui > .ui.gdpr.visible:upward(1)
hrkgame.com##html:root .modals.dimmer.ui > .ui.gdpr.hidden:upward(1):style(display: flex !important;)

Próbuję sprawdzać zmiany klasy .gdpr.visible <===> .gdpr.hidden

Sprawdzając - winny nie jest :upward ani :has. Pomysł z CSS działa przypadkowo w uBO,

AdGuard niestety filtr odrzuca jakby te pop-upy nie łapały się na testy watch-attr w ich silniku: https://ameshkov.github.io/web/watchattr.html (Edge i najnowszy dodatek AG oblewa test (lub się zdezaktualizował) - niby nie używa dodatek tabs.insertCSS()).

@gwarser
Copy link

gwarser commented Dec 10, 2020

@krystian3w ups...

[rdk@on uBlock-git]$ git bisect bad && ./tools/make-chromium.sh 
35aefed92616cbfb75f12f37c7ea7fb3a3cc3369 is the first bad commit
commit 35aefed92616cbfb75f12f37c7ea7fb3a3cc3369
Author: Raymond Hill <rhill@raymondhill.net>
Date:   Mon Sep 7 08:28:01 2020 -0400

    Add support to chain `:style()` to procedural operators
    
    Related issue:
    - https://github.com/uBlockOrigin/uBlock-issues/issues/382
    
    Additionally, remnant code for pseudo-user stylesheets
    has been removed. Related commit:
    - https://github.com/gorhill/uBlock/commit/5c68867b92735931a791dfedf4ef9608cc364862

 platform/chromium/vapi-background.js     |  23 ++-
 platform/chromium/vapi-client.js         |   8 +-
 src/js/background.js                     |   4 +-
 src/js/contentscript.js                  | 278 ++++++++++++-------------------
 src/js/cosmetic-filtering.js             | 167 ++++++-------------
 src/js/epicker-ui.js                     |  17 +-
 src/js/scriptlets/cosmetic-logger.js     |   6 +-
 src/js/scriptlets/dom-inspector.js       |  23 ++-
 src/js/scriptlets/dom-survey-elements.js |   4 +-
 src/js/scriptlets/epicker.js             | 180 ++++++--------------
 src/js/static-filtering-parser.js        |  43 +++--
 11 files changed, 281 insertions(+), 472 deletions(-)
*** uBlock0.chromium: Creating web store package
*** uBlock0.chromium: copying common files
*** Packaging assets in dist/build/uBlock0.chromium/assets... rm: cannot remove 'dist/build/uBlock0.chromium/assets/ublock/resources.txt': No such file or directory
done.
*** uBlock0.chromium: Generating meta...
*** uBlock0.chromium: Package done.
[rdk@on uBlock-git]$ 

@krystian3w
Copy link
Contributor

To naprawiamy czy je wygaszamy?

@gwarser
Copy link

gwarser commented Dec 10, 2020

Okazało się że w ogóle "odkrywanie" nie działa.

AdGuard zaimplementował watch-attr? Nie widzę dokumentacji w https://kb.adguard.com/en/general/how-to-create-your-own-ad-filters

@krystian3w
Copy link
Contributor

krystian3w commented Dec 10, 2020

Dodali niby natywne, ale musisz sprawdzać czy element ukrywa style="display: none !important;" czy tabs.insertCSS().

Jakieś tam wsparcie dla tabs.insertCSS() dodali dla AdGuard 3.3.2+.


Być może zachodzi u nich też jakaś regersja, że działa w Firefox ale już nie w Safari / Chromium / Edge / IE11.

@gwarser
Copy link

gwarser commented Dec 10, 2020

Na chrome nie działa. Z tego co rozumiem, filtry z #?# są dodawane w atrybucie style.

@krystian3w
Copy link
Contributor

Tak ale ExtendedCSS nie powinien być odrazu konwertowany na style, jak testowałem to z miesiąc temu w listopadzie to umiał dodatek używać tabs.insertCSS().

Domyślnie #?# powwino się przekształcać w atrybut tylko jak napotka prosty filtr:

example.org#?#h2

lub CSS:

example.org#$?#h2 { color: red }

@gwarser
Copy link

gwarser commented Dec 10, 2020

Ah, ok, właśnie widzę, że mi wstawia style. Ale AdGuard ma chyba identyczny błąd co uBO - jak kliknę drugi guzik to nic się nie dzieje, a dopiero jak kliknę pierwszy to się kwadrat pojawia. No chyba, że coś nie tak z moim testem jest.


W AdGuardzie testowane bez :watch-attr():

github.io,localhost###testdiv:has(p)
github.io,localhost###testdiv2:has(p)

@krystian3w

This comment has been minimized.

@gwarser
Copy link

gwarser commented Dec 10, 2020

@krystian3w ameshkov/web#2

Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants