Skip to content

Add more ad domains#3164

Merged
MkQtS merged 2 commits into
v2fly:masterfrom
DeepChirp:ads
Jan 11, 2026
Merged

Add more ad domains#3164
MkQtS merged 2 commits into
v2fly:masterfrom
DeepChirp:ads

Conversation

@DeepChirp

Copy link
Copy Markdown
Contributor

No description provided.

@MkQtS

MkQtS commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

新建 data/xxx-ads 然后在 data/xxxdata/category-ads 两处同时 include:xxx-ads,这样的操作应该不需要了

可以在 data/xxx 里直接添加带有 @ads 的域名,然后在 data/category-adsinclude:xxx @ads,这样可以减少 data 目录里的文件数量(当前的 main.go 也是支持的)

@DeepChirp

Copy link
Copy Markdown
Contributor Author

新建 data/xxx-ads 然后在 data/xxxdata/category-ads 两处同时 include:xxx-ads,这样的操作应该不需要了

可以在 data/xxx 里直接添加带有 @ads 的域名,然后在 data/category-adsinclude:xxx @ads,这样可以减少 data 目录里的文件数量(当前的 main.go 也是支持的)

确实,我以为得等到 #3119 合并之后才可以。我还把原先我自己做的部分拆分合并回去了,考虑到应该不会有人拦截单个分类的广告,我想应该是安全的

@MkQtS

MkQtS commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

picacg 那个带注释的域名建议不要动

@MkQtS

MkQtS commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

那把 #3040 引入的 github-ads 也处理掉吧

@DeepChirp

DeepChirp commented Jan 11, 2026

Copy link
Copy Markdown
Contributor Author

那把 #3040 引入的 github-ads 也处理掉吧

仿照Microsoft包含GitHub,我感觉在microsoft-ads里包含相关域名比较合适,而不是直接在category-ads中引入github @ads之类的。

@MkQtS

MkQtS commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

不好意思,刚才还有个问题没考虑到。现在的按属性选择性 include 是有问题的,不能递归,参考 #390 (comment)

这会导致 category-ads 无法包含 baidu-ads 里的 include:zuoyebang @ads 及 microsoft-ads 里的 include:github @ads,对于这两个,还是改回去吧

Co-authored-by: MkQtS <81752398+MkQtS@users.noreply.github.com>
@DeepChirp

Copy link
Copy Markdown
Contributor Author

不好意思,刚才还有个问题没考虑到。现在的按属性选择性 include 是有问题的,不能递归,参考 #390 (comment)

这会导致 category-ads 无法包含 baidu-ads 里的 include:zuoyebang @ads 及 microsoft-ads 里的 include:github @ads,对于这两个,还是改回去吧

好的,我感觉可以等到对应的PR合并后,再对相关的架构做调整

@MkQtS MkQtS merged commit 8786ff7 into v2fly:master Jan 11, 2026
1 check passed
@DeepChirp DeepChirp deleted the ads branch January 11, 2026 14:01
github-actions Bot added a commit to YW5vbnltb3Vz/domain-list-community that referenced this pull request Jan 11, 2026
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.

2 participants