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

https://github.com/easylist/easylist/issues/12147#issue-1251695641 #13443

Merged
merged 8 commits into from
May 29, 2022
Merged

https://github.com/easylist/easylist/issues/12147#issue-1251695641 #13443

merged 8 commits into from
May 29, 2022

Conversation

MasterKia
Copy link
Member

easylist/easylist#12147 (comment)
Since EasyList maintainer @Khrin didn't like cooperating with the downstream.

All rules should be kept until PersianBlocker is added as default (hopefully).
The only rules which are too important and can't be removed are (until EasyList decides to add them):

tamin.ir#@#.ads2
@@||tamin.ir/content/ads/

easylist/easylist#12147 (comment)
Since EasyList maintainer @Khrin didn't like cooperating with the downstream.

All rules should be kept *until* PersianBlocker is added as default (hopefully).
The only rules which are too important and can't be removed are (until EasyList decides to add them):
```
tamin.ir#@#.ads2
@@||tamin.ir/content/ads/
```
@Yuki2718
Copy link
Contributor

webii.ir##.ads-box appear to be indeed ads: https://www.webii.ir/cat/digital

@MasterKia
Copy link
Member Author

MasterKia commented May 29, 2022

##.ads-box

image

The website title is وب سایت تبلیغات و نیازمندیها درج آگهی رایگان which roughly translates to Free advertising (and selling stuff) website where you can submit your goods to sell.
Similar to eBay I'd say.

@Yuki2718
Copy link
Contributor

Hmm, so the website itself is for ads. I'm not sure if we should unblock. Similar to #10149 (comment) , user can turn blocker off on such sites. @mapx- @okiehsch

@MasterKia
Copy link
Member Author

Hmm, so the website itself is for ads. I'm not sure if we should unblock. Similar to #10149 (comment) , user can turn blocker off on such sites. @mapx- @okiehsch

It's not for ads per say, is eBay for ads where people can sell their own stuff?

@Yuki2718
Copy link
Contributor

Yuki2718 commented May 29, 2022

@@1000site.ir/home/ads/asset/favicon.ico unblocks ad too but it's addressed by your list (1000site.ir##.col-md-offset-6.col-md-3) and current IRN list (.ir/*/banner/). However, such cases are unlikely to be addressed by EL if the breakage is acceptable. Anyway you forgot ||.

MasterKia pushed a commit to MasterKia/PersianBlocker that referenced this pull request May 29, 2022
@Yuki2718
Copy link
Contributor

sourceiran.com/wp-content/uploads/300-250.jpg is self promotion and it's fair not to unblock:
https://easylist.to/pages/policy.html

Self-promotion should not be specifically removed by EasyList, although equally should not be allowed if it is blocked.

https://github.com/uBlockOrigin/uAssets

The rationale on whether to include a specific filter in one of uBO's own filter lists is the same as outlined by EasyList/EasyPrivacy policies

Anyway @@||sourceiran.com/wp-content/uploads/*.jpg alone doesn't work, sourceiran.com#@#.ads-widget is also needed - kinda too much to unblock single self-promo.

@MasterKia
Copy link
Member Author

I'll take back sourceiran.com from uBlock unbreak but can I address it in PersianBlocker?

@Yuki2718
Copy link
Contributor

I'll take back sourceiran.com from uBlock unbreak but can I address it in PersianBlocker?

Why not as that's your list?

@MasterKia
Copy link
Member Author

sourceiran.com#@#.ads-widget is also needed

BTW I had added this in my list, but forgot to include it in this PR.

@MasterKia
Copy link
Member Author

Similar to https://github.com/uBlockOrigin/uAssets/pull/10149#issuecomment-951821141

On that note, I noticed that clickyab.com (a Persian site dedicated to ads) is blocked in Pete Lowe's list and I did this:

@@||clickyab.com^$1p
||clickyab.com^$3p

Is this a wrong approach according to that comment from gorhill?

@mapx-
Copy link
Contributor

mapx- commented May 29, 2022

usually
||clickyab.com^$badfilter

and not @@||clickyab.com^$1p

@Yuki2718
Copy link
Contributor

cooldl.net#@#.ads-box
cooldl.net#@#.ads-content

self-promo too

@Yuki2718
Copy link
Contributor

@@||beytoote.com/ads/images_pc/$image
@@||beytoote.com/ads/images_mob/$image

merge to single rule @@||beytoote.com/ads/images_$image,1p

MasterKia pushed a commit to MasterKia/PersianBlocker that referenced this pull request May 29, 2022
@MasterKia
Copy link
Member Author

MasterKia commented May 29, 2022

usually ||clickyab.com^$badfilter

and not @@||clickyab.com^$1p

Fixed, I understand the concept of "badfilter" but does do any harm when unblocked using ^$1p?

@Yuki2718
Copy link
Contributor

Fixed, I understand the concept of "badfilter" but does do any harm when unblocked using ^$1p?

Depends on site but can allow too much - there may be something to be blocked by generic rules.

@Yuki2718
Copy link
Contributor

Yuki2718 commented May 29, 2022

@@||beytoote.com/ads/js/$script,1p will be needed too (bottom slider broken).

@MasterKia
Copy link
Member Author

Also @@||ads.beytoote.com/live_chat, breaks the chat function in beytoote.com/ads/advertising.php.

MasterKia pushed a commit to MasterKia/PersianBlocker that referenced this pull request May 29, 2022
@Yuki2718
Copy link
Contributor

Yuki2718 commented May 29, 2022

Lastly can you take more detailed screenshot(s) of tamin.ir (DOM tree)? Some government sites actually deliver ads. Want to confirm they're not ads.

@MasterKia
Copy link
Member Author

image

@Yuki2718
Copy link
Contributor

No need, so they're links to subpages of Iranian Social Security Organization.

@Yuki2718 Yuki2718 merged commit 26a9643 into uBlockOrigin:master May 29, 2022
@MasterKia MasterKia deleted the patch-1 branch May 29, 2022 05:36
@Yuki2718
Copy link
Contributor

7edc8a7

@Yuki2718
Copy link
Contributor

Adjusted a bit: 40a47e0 I'll copy them to AG list.

@MasterKia
Copy link
Member Author

MasterKia commented May 29, 2022

Should I keep them or remove them?
PersianBlocker if used with uBO and AG should be fine since these unbreak filters are present in both.

But if used with ABP (hopefully not :p), I guess they don't have an unbreak list and rely on EasyList.

@Yuki2718
Copy link
Contributor

We want to keep minimal rules in our lists.

Yuki2718 added a commit to AdguardTeam/AdguardFilters that referenced this pull request May 29, 2022
MasterKia pushed a commit to MasterKia/PersianBlocker that referenced this pull request May 29, 2022
@MasterKia
Copy link
Member Author

easylist/easylist#12152 (comment)

@ryanbr

Could you please tell me how that qualifies as spam?
I don't understand this hostility from @Khrin, the EL maintainer.
Does he have a problem with Persian websites?

MasterKia pushed a commit to MasterKia/PersianBlocker that referenced this pull request May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants