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

fix(route): bilibili add author to title #8220

Closed
wants to merge 1 commit into from
Closed

fix(route): bilibili add author to title #8220

wants to merge 1 commit into from

Conversation

Azureki
Copy link
Contributor

@Azureki Azureki commented Sep 20, 2021

该 PR 相关 Issue / Involved issue

无。

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

/bilibili/followings/dynamic/2267573

新RSS检查列表 / New RSS Script Checklist

  • New Route
  • Documentation
    • CN
    • EN
  • 全文获取 fulltext
    • Use Cache
  • 反爬/频率限制 anti-bot or rate limit?
    • 如果有, 是否有对应的措施? If yes, do your code reflect this sign?
  • 日期和时间 date and time
    • 可以解析 Parsed
    • 时区调整 Correct TimeZone
  • 添加了新的包 New package added
  • Puppeteer

说明 / Note

参考 weibo 时间线 和 twitter 时间线 的标题，添加了作者名用来区分动态。

路由 /bilibili/followings/dynamic/:uid 需 cookie 访问。

@Azureki
fix(route): bilibili add author to title
977925b 
参考 weibo 和 twitter 的时间线标题，添加了作者名用来区分动态。
@github-actions
Copy link
Contributor

Successfully generated as following:

https://rsshub-5052nraxo-diy.vercel.app/bilibili/followings/dynamic/2267573 - **Failed** 
    HttpError: <!DOCTYPE html><html><head><title>Error in RSSHub!</title><style>body{font-family:-apple-system,BlinkMacSystemFont,Segoe UI,Roboto,Oxygen,Ubuntu,Cantarell,Fira Sans,Droid Sans,Helvetica Neue,sans-serif;-webkit-font-smoothing:antialiased;-moz-osx-font-smoothing:grayscale;font-size:16px;color:#2c3e50}.content{width:800px;margin:100px auto 0;text-align:center}.message{white-space:pre-wrap;text-align:left}a{color:#f5712c}</style></head><body><div class="content"><p><img src="https://i.loli.net/2019/09/04/lmBbNQoLzrKGgiM.jpg" alt="RSSHub" width="120"></p><h1>Looks like something went wrong</h1><pre class="message">Route requested: /bilibili/followings/dynamic/2267573</pre><pre class="message">Error message: 缺少对应 uid 的 Bilibili 用户登录后的 Cookie 值</pre><div style="margin:50px 0"><p>在线文档与支持，请访问<a href="https://docs.rsshub.app/" target="_blank">docs.rsshub.app</a>。</p><p>For online documentation and support please refer to <a href="https://docs.rsshub.app/" target="_blank">docs.rsshub.app</a>.</p></div></div></body></html>

@github-actions github-actions bot added the Auto: Route Test Complete Auto route test has finished on given PR label Sep 20, 2021
@NeverBehave
Copy link
Collaborator

在除了作者属性以外的位置插入作者的信息不是一个正确的操作，其他路由的相关写法可能是因为历史遗留问题没有处理，这种操作不应当被继续推广

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Auto: Route Test Complete Auto route test has finished on given PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Azureki @NeverBehave