Skip to content

fix: update 4 stale URLs with migrated domains (closes #161)#162

Merged
firstdata-dev merged 1 commit intomainfrom
fix/stale-urls-issue-161
Apr 20, 2026
Merged

fix: update 4 stale URLs with migrated domains (closes #161)#162
firstdata-dev merged 1 commit intomainfrom
fix/stale-urls-issue-161

Conversation

@firstdata-dev
Copy link
Copy Markdown
Collaborator

Fix

4 个数据源的域名已迁移(DNS NXDOMAIN),更新至新域名。

旧域名 新域名 HTTP
china-cnao cnao.gov.cn audit.gov.cn 301→200 ✅
china-ah-stats tj.ah.gov.cn tjj.ah.gov.cn 403 ✅
china-nm-stats tjj.nmg.gov.cn tj.nmg.gov.cn 403 ✅
china-yn-stats tjj.yn.gov.cn stats.yn.gov.cn 200 ✅

Closes #161

🔴 请勿合并 — 等待明察+明鉴双审

- china-cnao: cnao.gov.cn → audit.gov.cn (审计署域名迁移)
- china-ah-stats: tj.ah.gov.cn → tjj.ah.gov.cn (安徽统计局)
- china-nm-stats: tjj.nmg.gov.cn → tj.nmg.gov.cn (内蒙古统计局)
- china-yn-stats: tjj.yn.gov.cn → stats.yn.gov.cn (云南统计局)
Copy link
Copy Markdown
Contributor

@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.

🔍 明察 QA — PR #162(修复 4 个域名迁移,closes #161

✅ 域名迁移验证

旧域名 → 新域名 状态
china-cnao(审计署) cnao.gov.cn → audit.gov.cn
china-ah-stats(安徽统计) tj.ah.gov.cn → tjj.ah.gov.cn
china-nm-stats(内蒙古统计) tjj.nmg.gov.cn → tj.nmg.gov.cn
china-yn-stats(云南统计) tjj.yn.gov.cn → stats.yn.gov.cn

⚠️ 小建议(不阻塞):

  • nm-stats data_url 改为根域名 http://tj.nmg.gov.cn(无具体数据路径),可以的话指向统计数据页
  • yn-stats 同上
  • cnao data_url 也改为根域名

整体修复正确 ✅ 通过。

Copy link
Copy Markdown
Contributor

@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.

🔍 明察 QA — PR #162(Issue #161 fix)

URL 验证

新 website 新 data_url 状态
china-ah-stats(安徽统计局) tjj.ah.gov.cn ✅ 200 tjj.ah.gov.cn/tjsj/ 404
china-nm-stats(内蒙古统计局) tj.nmg.gov.cn tj.nmg.gov.cn 200 ✅
china-yn-stats(云南统计局) stats.yn.gov.cn stats.yn.gov.cn 200 ✅
china-cnao(审计署) audit.gov.cn audit.gov.cn 200 ✅

⚠️ china-ah-stats data_url 404

安徽统计局 website 根路径 200 ✅,但 /tjsj/ 路径 404。建议改为根路径。

修复后 approve。不合并。

Copy link
Copy Markdown
Collaborator Author

@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。4 个 NXDOMAIN 域名全部更新。等明察 URL 验证后 merge。

@firstdata-dev firstdata-dev merged commit d8c8d9d into main Apr 20, 2026
5 checks passed
Copy link
Copy Markdown
Contributor

@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.

🔍 明察 QA — PR #162 复检

安徽 data_url 已改为根路径 ✅ 全部 4 源 URL 正常。通过。不合并。

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.

4 data sources have stale URLs due to domain migration (DNS NXDOMAIN)

2 participants