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

feat(LocalFeedRepository): throttle feed extraction #6949

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

FineFindus
Copy link
Contributor

Throttles the local feed extration in-order to avoid rate-limiting. This is done, similary to NewPipe, by introducting an articificial delay every 50 fetched feeds of on average 1 second. These values may have to be changed in the future.

Ref: #6941 (comment)

@FineFindus FineFindus force-pushed the feat/throttle-feed-loading branch 2 times, most recently from aae5579 to 8acaa16 Compare January 13, 2025 16:05
@FineFindus FineFindus force-pushed the feat/throttle-feed-loading branch from 8acaa16 to e9cb3d4 Compare January 13, 2025 16:32
@Bnyro Bnyro force-pushed the feat/throttle-feed-loading branch from e9cb3d4 to d614588 Compare January 13, 2025 16:39
Throttles the local feed extration in-order to avoid rate-limiting.
This is done, similary to NewPipe, by introducting an articificial
delay every 50 fetched feeds of on average 1 second. These values
may have to be changed in the future.

Ref: libre-tube#6941 (comment)
@Bnyro Bnyro force-pushed the feat/throttle-feed-loading branch from d614588 to 4b04932 Compare January 13, 2025 16:41
Copy link
Member

@Bnyro Bnyro left a comment

Choose a reason for hiding this comment

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

Thank you!

@Bnyro Bnyro merged commit 542963a into libre-tube:master Jan 13, 2025
1 of 2 checks passed
@FineFindus FineFindus deleted the feat/throttle-feed-loading branch January 13, 2025 16:53
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.

2 participants