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

Create fatal_http_errors.py patch #852

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Create fatal_http_errors.py patch #852

wants to merge 4 commits into from
+26 −0

Conversation

tcely
Copy link
Contributor

@tcely tcely commented Mar 16, 2025 •
edited
Loading

To respect HTTP 429 errors, we need to provide additional arguments to a function called in the YoutubeIE class.

Fixes #849

@tcely
Create fatal_http_errors.py
3d4d965 
This is a sketch of how a new patch is implemented.
It doesn't do anything yet.
@tcely tcely changed the title Create fatal_http_errors.py patch Create fatal_http_errors.py patch Mar 16, 2025
@tcely tcely marked this pull request as ready for review March 16, 2025 15:50
@tcely
Copy link
Contributor Author

tcely commented Mar 16, 2025 •
edited
Loading

This patch is working to cause tasks to be rescheduled.

Downloading metadata for "1cda2cbb-0048-49c4-9b53-fb3389207314", attempted 1 time
Error: "Failed to extract_info for "https://www.youtube.com/watch?v=KEY":
  ERROR: [youtube] KEY: Unable to download webpage:
  HTTP Error 429: Too Many Requests (caused by <HTTPError 429: Too Many Requests>)"
Task will be retried at 2025-03-16 19:28:43

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
None yet
Projects
Status
Status: Ready
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fail a task on 429 errors
1 participant
@tcely