Skip to content

feat: 增加 Hacker News 内容#2253

Merged
DIYgod merged 3 commits intoDIYgod:masterfrom
cf020031308:master
May 29, 2019
Merged

feat: 增加 Hacker News 内容#2253
DIYgod merged 3 commits intoDIYgod:masterfrom
cf020031308:master

Conversation

@cf020031308
Copy link
Copy Markdown
Contributor

No description provided.

@DIYgod
Copy link
Copy Markdown
Owner

DIYgod commented May 29, 2019

建议做得更通用一些,比如 /hackernews/:type

@cf020031308
Copy link
Copy Markdown
Contributor Author

cf020031308 commented May 29, 2019

可以。但这个加了之后怎么在本地看运行是否正常?比如怎么让将新加的内容打印出来我好先看看?

@cf020031308
Copy link
Copy Markdown
Contributor Author

试着 npm run dev 跑起来,但是请求时报错

error: Error in /hackernews/best/comments: Error: write EPROTO 4467213760:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:252:

    at WriteWrap.afterWrite [as oncomplete] (net.js:779:14)

@cf020031308
Copy link
Copy Markdown
Contributor Author

试着 npm run dev 跑起来,但是请求时报错

error: Error in /hackernews/best/comments: Error: write EPROTO 4467213760:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:252:

    at WriteWrap.afterWrite [as oncomplete] (net.js:779:14)

这是我本地开了代理的原因,关掉后就可以了。


已经扩展了分类。

本地钩子测试和人肉测试内容没发现问题。npm run test 时有个 cache 测试没过,原因是网站要 7、8 秒才返回结果,超过了配置的 5 秒,但这没办法改善。

麻烦 @DIYgod 看看。

@cf020031308 cf020031308 changed the title 增加 Hacker News 的热门内容 增加 Hacker News 内容 May 29, 2019
@DIYgod DIYgod changed the title 增加 Hacker News 内容 feat: 增加 Hacker News 内容 May 29, 2019
@DIYgod DIYgod merged commit 7ac9ee0 into DIYgod:master May 29, 2019
@DIYgod
Copy link
Copy Markdown
Owner

DIYgod commented May 29, 2019

Merged, thanks!

iplusx added a commit to iplusx/RSSHub that referenced this pull request May 31, 2019
* master: (47 commits)
  feat: Bilibii 关注专栏 (DIYgod#2260)
  feat: WIRED杂志根据标签订阅 (DIYgod#2262)
  docs: fix typo
  fix: douban movie
  feat: remove weibo/user2, close DIYgod#2169
  fix: notch time (DIYgod#2259)
  fix: 鼠绘无法获取最新漫画bug (DIYgod#2257)
  feat: 增加 Hacker News 内容 (DIYgod#2253)
  feat: 添加了移动支付网分类 (DIYgod#2247)
  docs: 好奇怪文档示例错误 (DIYgod#2255)
  feat: 增加好奇怪 - 好奇心日报荣誉出品 (DIYgod#2252)
  feat: 添加新浪专栏-创事记 (DIYgod#2251)
  feat:添加cnBeta最新文章 (DIYgod#2249)
  docs: format
  docs: update qa
  fix: 小黑盒热门折扣 (DIYgod#2237)
  docs: update aqicn.org 文档说明 (DIYgod#2240)
  fix: fix saraba1st docs (DIYgod#2238)
  feat: 增加猫眼电影 (DIYgod#2234)
  fix: 修复 语雀知识库link 404报错 (DIYgod#2230)
  ...

# Conflicts:
#	lib/routes/twitter/user.js
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