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(1lou): #18351 BT 之家 1LOU 站 #18366

Closed
wants to merge 3 commits into from
Closed

Conversation

falling
Copy link
Contributor

@falling falling commented Feb 15, 2025

Involved Issue / 该 PR 相关 Issue

Close #18351

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

NOROUTE

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

  • [ x] New Route / 新的路由
  • 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 / 说明

BT 之家 1LOU 站对于搜索有限制,需要添加User-Agent信息才能正常获取返回内容

@github-actions github-actions bot added Route Auto: Route Test Skipped PR involves no routes labels Feb 15, 2025
@falling falling changed the title fix issue #18351 BT 之家 1LOU 站 fix(1lou): #18351 BT 之家 1LOU 站 Feb 15, 2025
Copy link
Collaborator

@TonyRL TonyRL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since auto route test is skipped, we can't know if the issue is fixed or not.

const { data: response } = await got(currentUrl);
const { data: response } = await got(currentUrl, {
headers: {
'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/99.0.4844.51 Safari/537.36',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSSHub will use a randomised UA of Chrome on Mac. Does the site only work with this fixed version of Chrome on Windows?

@falling falling closed this Feb 16, 2025
@yishunzhikong
Copy link

你更改过后依然不能搜索,我尝试本地docker部署,也不行

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BT之家抓取不到了
3 participants