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

fix: overwrite url-test interval not work #3273

Merged
merged 1 commit into from
May 15, 2023
Merged

Conversation

cnsilvan
Copy link
Contributor

解决覆写URL-Test 策略组切换灵敏度(ms)、测速(连通性)间隔修改(s) 不生效的问题。
时间不够,没去研究clash源码,大概率是这个问题。由于加了 ‘ ’ 导致读取yaml时类型不对,导致失效。会出现每次url-test都会切换节点。网络就不稳定了。

@cnsilvan cnsilvan changed the title Update yml_rules_change.sh fix: overwrite url-test interval not work May 14, 2023
@cnsilvan
Copy link
Contributor Author

找到源码了。这里并不会走到ok去,因为类型转换失败了。相当于tolerance等于0了,这不就每次url-test都会切换了嘛。

func parseURLTestOption(config map[string]any) []urlTestOption {
	opts := []urlTestOption{}

	// tolerance
	if tolerance, ok := config["tolerance"].(int); ok {
		opts = append(opts, urlTestWithTolerance(uint16(tolerance)))
	}

	return opts
}

@vernesong
Copy link
Owner

thanks

1 similar comment
@vernesong
Copy link
Owner

thanks

@vernesong vernesong changed the base branch from master to dev May 15, 2023 00:12
@vernesong vernesong merged commit 0e8684d into vernesong:dev May 15, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request May 19, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request Jun 22, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request Jun 25, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request Jun 28, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request Jul 2, 2023
zheshifandian pushed a commit to zheshifandian/OpenClash that referenced this pull request Jul 6, 2023
oppen321 pushed a commit to oppen321/OpenClash that referenced this pull request Sep 21, 2024
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