Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

中山大学数据科学与计算机学院页面抓取失败,可能是学院网页升级改版造成的 #2905

Closed
Neutrino3316 opened this issue Aug 21, 2019 · 0 comments · Fixed by #2906
Labels
RSS bug Something isn't working

Comments

@Neutrino3316
Copy link
Contributor

路由地址

中山大学 数据科学与计算机学院
https://rsshub.app/sysu/sdcs

预期是什么?

预期:应该能抓取到学院的通知并返回xml文件

实际发生了什么?

页面并未成功返回xml文件,抓取失败。报错信息见额外信息。

额外信息(日志、报错等)

RSSHub 发生了一些意外:

TypeError: Cannot read property '1' of null
    at getDetail (/app/lib/routes/universities/sysu/sdcs.js:52:46)
    at runMicrotasks ()
    at processTicksAndRejections (internal/process/task_queues.js:85:5)
    at async Object.app.context.cache.tryGet (/app/lib/middleware/cache.js:136:17)
    at async Promise.all (index 1)
    at async module.exports (/app/lib/routes/universities/sysu/sdcs.js:24:15)
    at async cache (/app/lib/middleware/cache.js:260:9)
    at async module.exports (/app/lib/middleware/parameter.js:8:5)
    at async module.exports (/app/lib/middleware/template.js:15:5)
    at async /app/lib/middleware/api-response-handler.js:143:9
    at async module.exports (/app/lib/middleware/api-template.js:2:5)
    at async module.exports (/app/lib/middleware/utf8.js:4:5)
    at async module.exports (/app/lib/middleware/debug.js:14:5)
    at async module.exports (/app/lib/middleware/access-control.js:36:13)
    at async module.exports (/app/lib/middleware/header.js:20:5)
    at async module.exports (/app/lib/middleware/onerror.js:5:9)

可能的原因

学院官网升级改版了,旧的抓取规则不再适用。

@DIYgod DIYgod added the RSS bug Something isn't working label Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RSS bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants