Skip to content

Replace 100MB Cloudflare speedtest URL with 99MB#9076

Merged
2dust merged 1 commit into
2dust:masterfrom
taskmgr818:master
Apr 11, 2026
Merged

Replace 100MB Cloudflare speedtest URL with 99MB#9076
2dust merged 1 commit into
2dust:masterfrom
taskmgr818:master

Conversation

@taskmgr818
Copy link
Copy Markdown
Contributor

@taskmgr818 taskmgr818 commented Apr 11, 2026

fix #9071
找了半天别的测速源,最后发现改成 99999999 就行……

curl -I "https://speed.cloudflare.com/__down?bytes=99999999"
# HTTP/1.1 200 OK

Copilot AI review requested due to automatic review settings April 11, 2026 01:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses Issue #9071 where the Cloudflare 100MB speedtest URL returns 403 Forbidden without a Referer header, by switching the 100MB test source to Cachefly to keep the speed test feature working without adding headers across the codebase.

Changes:

  • Added Cachefly 100MB test URL to the SpeedTestUrls list.
  • Removed Cloudflare 100MB URL (bytes=100000000) from the SpeedTestUrls list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 141 to 147
public static readonly List<string> SpeedTestUrls =
[
@"https://cachefly.cachefly.net/50mb.test",
@"https://cachefly.cachefly.net/100mb.test",
@"https://speed.cloudflare.com/__down?bytes=10000000",
@"https://speed.cloudflare.com/__down?bytes=50000000",
@"https://speed.cloudflare.com/__down?bytes=100000000",
];
Copy link

Copilot AI Apr 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the Cloudflare 100MB URL from SpeedTestUrls won’t fix existing users who already have SpeedTestItem.SpeedTestUrl persisted as https://speed.cloudflare.com/__down?bytes=100000000; ConfigHandler.LoadConfig() only sets a default when the value is empty, so those configs will still hit 403 and speedtest will keep failing. Consider adding a small migration/normalization step during config load (e.g., if the saved URL equals the removed Cloudflare 100MB URL, replace it with the new Cachefly 100MB URL) so the bug is resolved without requiring manual user intervention.

Copilot uses AI. Check for mistakes.
@taskmgr818 taskmgr818 changed the title Replace 100MB Cloudflare speedtest URL with Cachefly Replace 100MB Cloudflare speedtest URL with 99MB Apr 11, 2026
@2dust 2dust merged commit 96e5c11 into 2dust:master Apr 11, 2026
@yfdyh000
Copy link
Copy Markdown
Contributor

批量测速会触发HTTP 429错误。

@2dust
Copy link
Copy Markdown
Owner

2dust commented Apr 16, 2026

批量测速会触发HTTP 429错误。

有什么解决办法?

@yfdyh000
Copy link
Copy Markdown
Contributor

批量测速会触发HTTP 429错误。

有什么解决办法?

不清楚,只是昨天看到了一次。似乎无法稳定重现。

@AoiNyanko
Copy link
Copy Markdown

批量测速会触发HTTP 429错误。

有什么解决办法?

不清楚,只是昨天看到了一次。似乎无法稳定重现。

加個Ref RequestHeader就能繼續100MB了, 很神奇

@yfdyh000
Copy link
Copy Markdown
Contributor

批量测速会触发HTTP 429错误。

有什么解决办法?

https://sea-repo.hostwinds.net/tests/100mb.zip 可以用。但如果放到软件里,我不确定能撑多久。

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.

[Bug]: Cloudflare 100MB 测速链接失效 (403)

5 participants