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.

Sign up for GitHub

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

Jump to bottom

修复懂球帝早报部分链接 404 的问题 #1498

Merged
merged 3 commits into from Feb 13, 2019
Merged

修复懂球帝早报部分链接 404 的问题 #1498

merged 3 commits into from Feb 13, 2019

Conversation

HenryQW
Copy link
Collaborator

@HenryQW HenryQW commented Jan 30, 2019

No description provided.

@DIYgod
Copy link
Owner

DIYgod commented Jan 31, 2019

为啥换了 node-fetch...

@HenryQW
Copy link
Collaborator Author

HenryQW commented Jan 31, 2019

@DIYgod axios 404 直接 throw exception，抓不了 status code

@DIYgod
Copy link
Owner

DIYgod commented Jan 31, 2019

@HenryQW try catch 可以抓到，catch 的 event 里有

@DIYgod DIYgod merged commit 8080038 into DIYgod:master Feb 13, 2019
@DIYgod
Copy link
Owner

DIYgod commented Feb 13, 2019

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DIYgod DIYgod DIYgod approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@HenryQW @DIYgod