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 handling Googlevideo URLs returning redirects #18

Closed
Kethsar opened this issue May 3, 2021 · 1 comment
Closed

Fix handling Googlevideo URLs returning redirects #18

Kethsar opened this issue May 3, 2021 · 1 comment

Comments

@Kethsar
Copy link
Owner

Kethsar commented May 3, 2021

It seems direct googlevideo.com URLs can return plain text with intent to have you redirect, instead of redirecting you itself.

  • Check the content-type header for text/plain and assume it's a redirect URL.
  • Potentially recursively call the function that checks up to X times until a working URL is given.
  • Move the sq param to the end of the URL instead of truncating the rest of the URL. Could end up never retrieving video or audio.
@Kethsar
Copy link
Owner Author

Kethsar commented Dec 16, 2022

I've never seen this outside of the one time that caused me to open this ticket, and I think it may have been in part from trying to mess with the url while testing stuff in the browser.

@Kethsar Kethsar closed this as completed Dec 16, 2022
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

No branches or pull requests

1 participant