Skip to content

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

Closed
mingcha-dev wants to merge 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/update-redirect-urls
Closed

fix: update redirect URLs for china-robot-industry-alliance and cifar#83
mingcha-dev wants to merge 1 commit intoMLT-OSS:mainfrom
mingcha-dev:fix/update-redirect-urls

Conversation

@mingcha-dev
Copy link
Copy Markdown
Contributor

Summary

Update 2 data source URLs to eliminate unnecessary redirects.

Changes

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 URL triggers 301 redirect, adding latency and potential cross-protocol issues for Java clients.

cifar.json

  • website: https://www.cs.toronto.eduhttps://web.cs.toronto.edu
  • Reason: www.cs.toronto.edu has migrated to web.cs.toronto.edu (302 redirect). data_url unchanged as it resolves directly without redirect.

Verification

URL Status Redirects
https://cria.mei.net.cn/ ✅ 200 0
https://cria.mei.net.cn/gzpt.asp?lm=/1310 ✅ 200 0
https://web.cs.toronto.edu ✅ 200 0

Context

Found during URL verification study (redirect chain analysis). Ref: #qa-dns thread.

- china-robot-industry-alliance: http → https (301 redirect, site supports HTTPS)
- cifar: www.cs.toronto.edu → web.cs.toronto.edu (302 domain migration)

Verified both URLs return 200 after update.
Reduces redirect hops and avoids potential cross-protocol issues.
Copy link
Copy Markdown
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. HTTP→HTTPS + 301 重定向修复,好清理。

Copy link
Copy Markdown
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: HTTP to HTTPS and redirect URL fixes, both valid. LGTM

@mingcha-dev
Copy link
Copy Markdown
Contributor Author

Splitting into separate PR (cifar only) and issue (china-robot-industry-alliance investigation). See #84 and new issue.

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