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

Advertisements: ygosu.com inven.co.kr psss.kr [ Ad-Shield ] #294

Open
ghost opened this issue Jan 19, 2022 · 18 comments
Open

Advertisements: ygosu.com inven.co.kr psss.kr [ Ad-Shield ] #294

ghost opened this issue Jan 19, 2022 · 18 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jan 19, 2022

Summary

On Mozilla Firefox with AdGuard Extra (Tampermonkey), advertisements on ppss.kr, ygosu.com and maple.inven.co.kr are not hidden.

URLs

https://ygosu.com/
https://ygosu.com/community/news/14420
https://inven.co.kr/
https://www.inven.co.kr/webzine/news/?news=267624
https://ppss.kr/
https://ppss.kr/archives/247639

Screenshots

ygosu.com

Screenshot

inven.co.kr

Screenshot from 2022-01-19 16-18-27
Screensho1t
Screenshot

ppss.kr

Screenshot from 2022-01-19 16-23-06
Screenshot from 2022-01-19 16-23-49

Related Issue

#283
AdguardTeam/AdguardFilters#87725
AdguardTeam/AdguardFilters#87725
AdguardTeam/AdguardFilters#90634

@ghost
Copy link
Author

ghost commented Jan 25, 2022

@stanislav-atr AdguardTeam/AdguardFilters#108136

@ghost ghost changed the title Advertisements: ygosu.com maple.inven.co.kr psss.kr [ Ad-Shield ] Advertisements: ygosu.com inven.co.kr psss.kr [ Ad-Shield ] Jan 26, 2022
@ghost
Copy link
Author

ghost commented Jan 26, 2022

yous/YousList#203

@JobcenterTycoon

This comment was marked as outdated.

@Alex-302
Copy link
Member

@JobcenterTycoon From uBO? They will stop working after being added to our filter.

@JobcenterTycoon

This comment was marked as outdated.

@piquark6046
Copy link
Member

piquark6046 commented Feb 12, 2022

@JobcenterTycoon @Alex-302
The following rules on Mozilla Firefox (98.0b3) with AdGuard Browser Extension (4.0.108) can resolve partly:

@@/:\/\/ygosu\.com\/([a-z]|[A-Z])\w+\/([A-Z])\w+\.ps/
@@/:\/\/ygosu\.com\/([A-Z])\w+\.ps/
/:\/\/ygosu\.com\/([A-Z])\w+\./$object,image,media,other,redirect=32x32-transparent.png
/:\/\/ygosu\.com\/([a-z]|[A-Z])\w+\/([A-Z])\w+\./$object,image,media,other,redirect=32x32-transparent.png
Screenshot

Screenshot

List-KR/List-KR@26bba94
List-KR/List-KR@c995dcd


I am developing an userscript that can resolve this issue.

@piquark6046
Copy link
Member

Confirmed on Naver Whale, Microsoft Edge and Google Chrome.

Screenshots

Screenshot from 2022-02-12 18-04-45
Screenshot from 2022-02-12 18-06-43
Screenshot from 2022-02-12 18-07-40

@piquark6046
Copy link
Member

#294 (comment)
@yous Can you re-confirm it?

@yous
Copy link

yous commented Feb 13, 2022

I saw some exceptions:

  • /jzvRJbqIUzuGQkjTcmrafcQHymy_75ca6bc055ee9fdb.xlsx: filename starts with [a-z], contains _
  • /PghIA_LdJRYC/blaFCKPPxdad662928f853f65.docx: directory contains _, filename starts with [a-z]
  • /wQvsyxyxtfbpLSvSKeermqZ/q0bdb1d57063bfd1b.ps: directory starts with [a-z], filename starts with [a-z]
  • /_yZSMrzZOoElgrMmR_gsZAIAVJja834228cded555d79.tif: filename starts with _, but it's rare

So I think you can start from this:

@@/:\/\/ygosu\.com\/[A-Za-z][\w_]+\/[A-Za-z][\w_]+\.ps/
@@/:\/\/ygosu\.com\/[A-Za-z][\w_]+\.ps/
/:\/\/ygosu\.com\/[A-Za-z][\w_]+\./$object,image,media,other,redirect=32x32-transparent.png
/:\/\/ygosu\.com\/[A-Za-z][\w_]+\/[A-Za-z][\w_]+\./$object,image,media,other,redirect=32x32-transparent.png

piquark6046 added a commit to List-KR/List-KR that referenced this issue Feb 13, 2022
@piquark6046
Copy link
Member

@zloyden
Copy link

zloyden commented Mar 24, 2022

@stanislav-atr please also include this website tgd.kr AdguardTeam/AdguardFilters#111644

@zloyden
Copy link

zloyden commented Apr 8, 2022

loawa.com AdguardTeam/AdguardFilters#114791

@Alex-302
Copy link
Member

Alex-302 commented May 9, 2022

@JobcenterTycoon
Copy link

There is now a great userscript which defuse the Ad-Shield crap completely https://github.com/List-KR/microShield

@JobcenterTycoon
Copy link

@stanislav-atr will microShield be implemented?

@piquark6046
Copy link
Member

@stanislav-atr @JobcenterTycoon adshield-defuser, original source code of microShield, is now open-source.

@JobcenterTycoon
Copy link

There are over 25 sites which using this now:

loawa.com
ygosu.com
sportalkorea.com
algumon.com
hancinema.net
enetnews.co.kr
edaily.co.kr
economist.co.kr
etoday.co.kr
hankyung.com
isplus.com
hometownstation.com
kagit.kr
inven.co.kr
viva100.com
joongdo.co.kr
jjang0u.com
tenbizt.com
tvreport.co.kr
newautopost.co.kr
mememedia.co.kr
mobilitytv.co.kr
cboard.net
a-ha.io
interfootball.co.kr
fourfourtwo.co.kr

The fix for ppss.kr is obsolete because ppss dropped the script.

@piquark6046
Copy link
Member

The microShield userscript will not be implemented in AdGuard Extra because it is too big to be implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants