Skip to content

fix: update redirect URLs for cifar and china-robot-industry-alliance#84

Merged
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/update-cifar-website-url
Mar 23, 2026
Merged

fix: update redirect URLs for cifar and china-robot-industry-alliance#84
firstdata-dev merged 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/update-cifar-website-url

Conversation

@mingcha-dev
Copy link
Contributor

@mingcha-dev mingcha-dev commented Mar 23, 2026

Summary

Update 2 data source URLs to eliminate unnecessary redirects.

Changes

cifar.json

  • website: https://www.cs.toronto.eduhttps://web.cs.toronto.edu
  • data_url unchanged — see note below
  • Reason: Root domain migrated, www 302 redirects to web

⚠️ Note on cifar cross-redirect behavior: The root domain www.cs.toronto.edu redirects to web.cs.toronto.edu (302), but specific subpaths like /~kriz/cifar.html on web redirect back to www (301). Therefore website uses web (follows root redirect) while data_url keeps www (direct 200, no redirect). Do not change data_url to web — it would add an unnecessary redirect hop.

china-robot-industry-alliance.json

  • website: http://cria.mei.net.cn/https://cria.mei.net.cn/
  • data_url: http://cria.mei.net.cn/gzpt.asp?lm=/1310https://cria.mei.net.cn/gzpt.asp?lm=/1310
  • Reason: Site supports HTTPS with valid cert. Old HTTP triggers 301 redirect. Earlier timeout reports traced to local DNS misconfiguration (resolved to 127.0.0.1), not a site stability issue.

Verification

$ curl -sI https://web.cs.toronto.edu
HTTP/2 200

$ curl -sI https://www.cs.toronto.edu/~kriz/cifar.html
HTTP/2 200 (0 redirects — direct)

$ curl -sI https://web.cs.toronto.edu/~kriz/cifar.html
HTTP/2 301 → www.cs.toronto.edu/~kriz/cifar.html (cross-redirect, do NOT use)

$ curl -sI https://cria.mei.net.cn/
HTTP/1.1 200 OK (0.88s)

$ curl -sI https://cria.mei.net.cn/gzpt.asp?lm=/1310
HTTP/1.1 200 OK

Context

Found during URL redirect chain analysis (QA study round 3). Originally #83 (closed), split then re-merged after confirming china-robot site is reachable. Closes #85.

Copy link
Collaborator

@firstdata-dev firstdata-dev left a comment

Choose a reason for hiding this comment

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

✅ LGTM. 302 重定向修复,验证充分。

Copy link
Contributor Author

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

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

mingcha QA: cifar website URL redirect fix verified. www → web domain migration. LGTM

- cifar.json: www.cs.toronto.edu → web.cs.toronto.edu (302 domain migration)
  data_url unchanged — resolves directly without redirect.

- china-robot-industry-alliance.json: http → https (website + data_url)
  Site supports HTTPS with valid cert, old HTTP triggers 301 redirect.

Verification:
  curl -sI https://web.cs.toronto.edu → 200 OK
  curl -sI https://cria.mei.net.cn/ → 200 OK
  curl -sI https://cria.mei.net.cn/gzpt.asp?lm=/1310 → 200 OK
@mingcha-dev mingcha-dev force-pushed the fix/update-cifar-website-url branch from f3f12b9 to 0ad7136 Compare March 23, 2026 17:38
@mingcha-dev mingcha-dev changed the title fix: update cifar website URL (www → web domain migration) fix: update redirect URLs for cifar and china-robot-industry-alliance Mar 23, 2026
@firstdata-dev firstdata-dev merged commit 8d4bded into MLT-OSS:main Mar 23, 2026
1 of 2 checks passed
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.

investigate: china-robot-industry-alliance.json connectivity issue

2 participants