Skip to content

Update ClashMeta.php#865

Closed
yootus wants to merge 1 commit into
cedar2025:masterfrom
yootus:patch-3
Closed

Update ClashMeta.php#865
yootus wants to merge 1 commit into
cedar2025:masterfrom
yootus:patch-3

Conversation

@yootus
Copy link
Copy Markdown
Contributor

@yootus yootus commented Apr 6, 2026

clash配置文件中第一行规则让订阅走直连,获取订阅的方式不对,目前是会获取到后台的域名。不太安全,改为直接获取后台设置的订阅域名

@darwinchow
Copy link
Copy Markdown

When multiple subscription domains are configured in the admin panel, Helper::getSubscribeUrl() returns one at random. As a result, the domain the user actually accesses may not match the one specified in the 'direct' rule, rendering that rule ineffective.

@yootus
Copy link
Copy Markdown
Contributor Author

yootus commented Apr 7, 2026

When multiple subscription domains are configured in the admin panel, Helper::getSubscribeUrl() returns one at random. As a result, the domain the user actually accesses may not match the one specified in the 'direct' rule, rendering that rule ineffective.

这个原本代码一直都是只返回一个的,如果多个订阅那就获取订阅的时候得返回所有的订阅链接了,直接使用 $urlString = (string)admin_setting('subscribe_url', '');
$subscribeUrlList = $urlString ? explode(',', $urlString) : []; 遍历这个list设置

@yootus yootus closed this by deleting the head repository May 22, 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