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

Rate Limit on RSS feeds #8350

Closed
Balthier opened this issue Feb 2, 2018 · 4 comments · Fixed by #19801
Closed

Rate Limit on RSS feeds #8350

Balthier opened this issue Feb 2, 2018 · 4 comments · Fixed by #19801
Assignees
Labels
Feature request Meta RSS RSS-related issues/changes

Comments

@Balthier
Copy link

Balthier commented Feb 2, 2018

qBittorrent version and Operating System

4.0.3 - Windows 10 Professional

At present it appears that qBittorent attempts to fetch all RSS feeds either at the same time, or in rapid succession.

This causes read errors when using multiple RSS feeds from a single site which imposes a connection limit.

Manually refreshing RSS feeds in smaller groups, works perfectly each time.

Please can we get an option which defines either/both of the following:
RSS connections per second
Number of RSS feeds to refresh per check

Either of those options would allow us to set the rate limit manually based on our individual requirements.

Duplicates and Related

#17867
#17856
#18671
#19619
#14689

@http403
Copy link

http403 commented Jul 21, 2020

This issue opened for more than 2 years already and I think it need some attention. I'm managing a small list of RSS feeds (20) and I already hitting website's rate limit. I really hope this feature got implemented.

@thalieht thalieht added the RSS RSS-related issues/changes label Jul 21, 2020
@sledgehammer999 sledgehammer999 changed the title Feature Request: Rate Limit on RSS feeds Rate Limit on RSS feeds Oct 29, 2020
@luzpaz luzpaz added the Meta label Sep 20, 2023
@jNullj
Copy link
Contributor

jNullj commented Oct 22, 2023

I might be able to help, i want to add a settings in the rss settings menu for a delay in seconds when calling the RSS feeds which is a simple solution but i doubt people use so many rss feeds that those few seconds would affect them.

I want to confirm before i setup the local dev env that:

  1. This solution is acceptable
  2. No one else started working on this, please assign me to let me know

@luzpaz
Copy link
Contributor

luzpaz commented Oct 23, 2023

@xavier2k6 ☝️

@glassez
Copy link
Member

glassez commented Oct 23, 2023

At present it appears that qBittorent attempts to fetch all RSS feeds either at the same time, or in rapid succession.

In fact, a "sequencing" mechanism was added some time ago to prevent simultaneous requests from multiple RSS feeds to the same site. But it has no additional delay. Requests are simply queued, and each subsequent one is executed as soon as the previous one is completed. So we could extend it by adding the ability to use a configurable delay.

jNullj added a commit to jNullj/qBittorrent that referenced this issue Oct 25, 2023
This commit only adds the RSS fetch delay setting to gui and loads it to to RssSession

Part 1 for qbittorrent#8350
jNullj added a commit to jNullj/qBittorrent that referenced this issue Oct 25, 2023
jNullj added a commit to jNullj/qBittorrent that referenced this issue Oct 27, 2023
Add delay to requests with the same ServiceID
The delay length is based on RSS fetch delay

Fixes qbittorrent#8350
jNullj added a commit to jNullj/qBittorrent that referenced this issue Dec 4, 2023
Update delay for registered sequential download services.
User might change rss fetch delay.
This commit updates existing services with new values
to take instant effect for the change without the need for a restart.

Closes qbittorrent#8350
glassez pushed a commit that referenced this issue Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Meta RSS RSS-related issues/changes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants