Replace 100MB Cloudflare speedtest URL with 99MB#9076
Conversation
There was a problem hiding this comment.
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
SpeedTestUrlslist. - Removed Cloudflare 100MB URL (
bytes=100000000) from theSpeedTestUrlslist.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 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", | ||
| ]; |
There was a problem hiding this comment.
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.
|
批量测速会触发HTTP 429错误。 |
有什么解决办法? |
不清楚,只是昨天看到了一次。似乎无法稳定重现。 |
加個Ref RequestHeader就能繼續100MB了, 很神奇 |
https://sea-repo.hostwinds.net/tests/100mb.zip 可以用。但如果放到软件里,我不确定能撑多久。 |
fix #9071
找了半天别的测速源,最后发现改成 99999999 就行……