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

fix(route): fix twitter error condition #13943

Merged
merged 1 commit into from
Dec 3, 2023

Conversation

TonyRL
Copy link
Collaborator

@TonyRL TonyRL commented Dec 3, 2023

Involved Issue / 该 PR 相关 Issue

Close #

Example for the Proposed Route(s) / 路由地址示例

/twitter/user/DIYgod

New RSS Route Checklist / 新 RSS 路由检查表

  • New Route / 新的路由
  • Documentation / 文档说明
  • Full text / 全文获取
    • Use cache / 使用缓存
  • Anti-bot or rate limit / 反爬/频率限制
    • If yes, do your code reflect this sign? / 如果有, 是否有对应的措施?
  • Date and time / 日期和时间
    • Parsed / 可以解析
    • Correct time zone / 时区正确
  • New package added / 添加了新的包
  • Puppeteer

Note / 说明

Copy link
Contributor

github-actions bot commented Dec 3, 2023

Successfully generated as following:

http://localhost:1200/twitter/user/DIYgod - Failed ❌
HTTPError: Response code 404 (Not Found)

Route requested: /user/DIYgod
Error message: Error: Invalid twitter oauth tokens
    at twitterGot (/app/lib/v2/twitter/web-api/twitter-api.js:13:15)
    at userByScreenName (/app/lib/v2/twitter/web-api/twitter-api.js:192:5)
    at userByAuto (/app/lib/v2/twitter/web-api/twitter-api.js:205:12)
    at /app/lib/v2/twitter/web-api/twitter-api.js:207:81
    at Object.tryGet (/app/lib/middleware/cache/index.js:64:27)
    at async getUser (/app/lib/v2/twitter/web-api/twitter-api.js:213:22)
    at async module.exports (/app/lib/v2/twitter/web-api/user.js:12:22)
    at async /app/node_modules/.pnpm/koa-mount@4.0.0/node_modules/koa-mount/index.js:58:5
    at async /app/node_modules/.pnpm/koa-mount@4.0.0/node_modules/koa-mount/index.js:52:26
    at async /app/lib/middleware/load-on-demand.js:29:9
Helpful Information to provide when opening issue:
Path: /user/DIYgod
Node version: v20.10.0
Git Hash: 343037a

@github-actions github-actions bot added the Auto: Route Test Complete Auto route test has finished on given PR label Dec 3, 2023
@TonyRL TonyRL merged commit c06353f into DIYgod:master Dec 3, 2023
35 checks passed
@TonyRL TonyRL deleted the fix/twitter/error branch December 3, 2023 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto: Route Test Complete Auto route test has finished on given PR Route: v2 v2 route related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant