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

[LBRY/Odysee] workaround for the 1000 item API-limit #10164

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Synarp
Copy link

@Synarp Synarp commented Jun 12, 2024

Description of your pull request and other information

The API used to enumerate videos on lbry/odysee is limited to 1000 Videos. I found a workaround for this problem here: https://www.reddit.com/r/lbry/comments/tpc7yz/get_list_of_all_videos_uploaded_by_user/

This works by filtering the videos such that the already enumerated ones are excluded.

If the videos are requested to be sorted newest first(which is the default), the workaround is used and otherwise a warning is emitted(yt-dlp silently limiting or not limiting the number of Videos it can find depending on the ordering would be confusing and unexpected).

Fixes #1296 (although this wasn't seen as a bug)

Template

Before submitting a pull request make sure you have:

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

  • I am the original author of this code and I am willing to release it under Unlicense
  • I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

@Synarp Synarp marked this pull request as ready for review June 12, 2024 00:58
@Synarp Synarp changed the title workaround for the 1000 item API-limit for lbry/odysee [LBRY/Odysee] workaround for the 1000 item API-limit for lbry/odysee Jun 12, 2024
@Synarp Synarp changed the title [LBRY/Odysee] workaround for the 1000 item API-limit for lbry/odysee [LBRY/Odysee] workaround for the 1000 item API-limit Jun 12, 2024
@bashonly bashonly added the site-enhancement Feature request for some website label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-enhancement Feature request for some website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Broken] Odysee: Only lists the last 1000 videos
2 participants