Skip to content

Update 18comic & picacg#2695

Merged
rootmelo92118 merged 3 commits intov2fly:masterfrom
Jiew-KueJwiaeng:Update
Dec 15, 2025
Merged

Update 18comic & picacg#2695
rootmelo92118 merged 3 commits intov2fly:masterfrom
Jiew-KueJwiaeng:Update

Conversation

@Jiew-KueJwiaeng
Copy link
Contributor

PR #2687

Sorry about that—I thought adding more attributes would be helpful (I even recall an old issue encouraging detailed attributes). I’ve just pushed a new PR—would love any feedback!


很抱歉,我以為這樣多加 attribute 是好的,我印象裡多年前也有 Issues 鼓勵詳加 attribute 。

現已提出新的 PR ,還望斧正。

@IceCodeNew
Copy link
Collaborator

很抱歉,我以為這樣多加 attribute 是好的,我印象裡多年前也有 Issues 鼓勵詳加 attribute 。

現已提出新的 PR ,還望斧正。

多加 attribute 是好的,只是烦请给出辅助判断的依据,这样 reviewer 才可以确定你加的 attribute 是对的。

@IceCodeNew IceCodeNew requested a review from rootmelo92118 May 19, 2025 09:53
@rootmelo92118 rootmelo92118 merged commit 6a11042 into v2fly:master Dec 15, 2025
1 check passed
github-actions bot added a commit to YW5vbnltb3Vz/domain-list-community that referenced this pull request Dec 15, 2025
* https://github.com/v2fly/domain-list-community:
  Add Ookla Speedtest Central to the list (v2fly#3053)
  Update 18comic & picacg (v2fly#2695)
  Chore: bump actions/upload-artifact from 5 to 6 (v2fly#3051)
@luochen1990
Copy link
Contributor

luochen1990 commented Dec 16, 2025

# Image Resource Domain like `img.diwodiwo.xyz` `s3.diwodiwo.xyz` `storage.diwodiwo.xyz` `storage-b.diwodiwo.xyz`
regexp:^([a-z0-9-]+\.)*(?!ad-display\.|ad-channel\.)[a-z0-9-]+\.diwodiwo\.xyz$

这段会导致某些软件(如dae)报错,希望能 revert。 或者加到单独的文件里,不要加到 geosite:geolocation-!cn 这个被广泛使用的分类里。

@luochen1990
Copy link
Contributor

初步判断应该是 (?! 这类复杂的 look ahead 正则导致的问题,建议用其他方式处理,避免用 这类正则

@AzunyanDaisuki
Copy link

初步判断应该是 (?! 这类复杂的 look ahead 正则导致的问题,建议用其他方式处理,避免用 这类正则

是的,look ahead 不被 Golang 支持,不符合本项目 https://github.com/v2fly/domain-list-community/blob/master/README.md 中对于语法的要求
Regular expression begins with `regexp:`, followed by a valid regular expression (per Golang's standard).

@95289527
Copy link

一条干崩多少客户端😂

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.

6 participants