Skip to content

feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播 with query string switch#6004

Closed
shunf4 wants to merge 19 commits intoDIYgod:masterfrom
shunf4:feat-more-readable-social
Closed

feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播 with query string switch#6004
shunf4 wants to merge 19 commits intoDIYgod:masterfrom
shunf4:feat-more-readable-social

Conversation

@shunf4
Copy link
Copy Markdown
Contributor

@shunf4 shunf4 commented Oct 26, 2020

目的 / What this PR does

  • 更可读的微博个人时间线、Twitter 用户时间线、豆瓣用户广播,通过 query string 开启,默认情况下变动不大
    • 豆瓣因必须修复 bug 内容有变动
  • 修复了微博在某些情况下的显示 bug、豆瓣内容解析错误、不全的 bug
  • 潜在地引入了新的 bug
  • 添加了关于如何使用这些 query string 的文档(中文:微博、豆瓣、Twitter;英文:Twitter)

更可读的微博小秘书 微博 RSS

更可读的 DIYgod Twitter RSS

更可读的豆瓣读书 RSS

该 PR 相关 Issue / Involved issue

完整路由地址 / Example for the proposed route(s)

  • /douban/people/113894409/status

  • /douban/people/113894409/status/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweet=1&showRetweetTextInTitle=1&addLinkForPics=1&showTimestampInDescription=1&showComments=1&widthOfPics=100

  • /twitter/user/durov

  • /twitter/user/durov/exclude_rts_replies

  • /twitter/user/durov/excludeReplies=1&includeRts=0&readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showQuotedAuthorAvatarInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweetAndReply=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showQuotedInTitle=1&heightOfPics=150

  • /twitter/keyword/推特/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showQuotedAuthorAvatarInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweetAndReply=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showQuotedInTitle=1&heightOfPics=150

  • /twitter/user/DIYgod/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showQuotedAuthorAvatarInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweetAndReply=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showQuotedInTitle=1&heightOfPics=150

  • /twitter/followings/DIYgod

  • /twitter/followings/DIYgod/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showQuotedAuthorAvatarInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweetAndReply=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showQuotedInTitle=1&heightOfPics=150

  • /weibo/user/1642909335

  • /weibo/user/1642909335/0

  • /weibo/user/1968133987/displayVideo=0&readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweet=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showTimestampInDescription=1&heightOfPics=150

  • /weibo/super_index/1008084989d223732bf6f02f75ea30efad58a9/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweet=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showTimestampInDescription=1&heightOfPics=150

  • /weibo/keyword/转发/readable=1&authorNameBold=1&showAuthorInTitle=1&showAuthorInDesc=1&showAuthorAvatarInDesc=1&showEmojiForRetweet=1&showRetweetTextInTitle=0&addLinkForPics=1&showTimestampInDescription=1&showTimestampInDescription=1&heightOfPics=150

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/diy/rsshub/5lc62o59d
✅ Preview: https://rsshub-git-feat-more-readable-social.diy.now.sh

@DIYgod
Copy link
Copy Markdown
Owner

DIYgod commented Oct 26, 2020

query string 是忽略缓存的,只用于全局参数,不应该用于路由参数

后果是

一个人先访问了 /douban/people/113894409/status?readable=1&authorNameBold=1,结果会被缓存起来
第二个人访问 /douban/people/113894409/status,就会直接返回前一个链接的结果

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 26, 2020

@DIYgod 我看到装备前线里用到了 query string 才这么写。请问这个是不是也有问题?

那我就仿照 arXiv 把参数放在 URL 里了,应该会导致缓存率下降,这个问题大不大

@DIYgod
Copy link
Copy Markdown
Owner

DIYgod commented Oct 26, 2020

@shunf4 是的,那个也有问题;问题不大

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 27, 2020

完工

注意默认情况下还是有些小修改的,比如 Twitter 的转推补充了信息,以及修复了微博、豆瓣的一些显示 bug 等,可能仍然会破坏一些人写的自动化机制

@HenryQW
Copy link
Copy Markdown
Collaborator

HenryQW commented Oct 29, 2020

@MisteryMonster @mashirozx @Rongronggg9 大家怎么看

@HenryQW
Copy link
Copy Markdown
Collaborator

HenryQW commented Oct 29, 2020

改动太大,3条路由必须分开提交

@MisteryMonster
Copy link
Copy Markdown
Contributor

因为没有用微博和 twitter 所以我只看了豆瓣的,感觉特别棒❤

@mashirozx
Copy link
Copy Markdown
Contributor

表面看起来这些选项挺实用的。

@Rongronggg9
Copy link
Copy Markdown
Contributor

看了微博的,应该没有什么问题。
感谢 @shunf4 的努力。

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 30, 2020

感谢,我另提三个 PR。

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.

7 participants