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

Fixed 404 response on age restricted videos #640

Closed
wants to merge 2 commits into from
Closed

Fixed 404 response on age restricted videos #640

wants to merge 2 commits into from

Conversation

Walkyst
Copy link
Contributor

@Walkyst Walkyst commented Jun 20, 2021

Now this YouTube endpoint also require provide client and his version, using here ANDROID because in response we receive always deciphered links so it won't give 403 errors

@Muh9049
Copy link

Muh9049 commented Jun 20, 2021

@Walkyst Just tried this but it's returning no results found for age-restricted videos.

For regular videos, it returns some weird error: https://srcb.in/rindzjCAlp,

Any idea why? Did they ban this method again within these 8 hours?

@Walkyst
Copy link
Contributor Author

Walkyst commented Jun 20, 2021

Any idea why? Did they ban this method again within these 8 hours?

Nah, it is because pbj requests require right headers

@AFKJacob
Copy link

why this is still not merged and why is the repo so inactive now? @sedmelluq
i have been having this issues for over a week now

@MCOfficer
Copy link
Contributor

MCOfficer commented Jun 22, 2021

why this is still not merged and why is the repo so inactive now? @sedmelluq
i have been having this issues for over a week now

Welcome to open source, where maintainers have a real life and obligations don't exist. If you urgently need it, the only real solution is to pay sed for finding time for it right now or fork.

@sedmelluq
Copy link
Owner

Adding yet another HTTP interface manager would require exposing it for configuring as well. Instead, changed the context filter on the main one to only add the PBJ headers for requests with &pbj=1 in their url.

@sedmelluq sedmelluq closed this Jun 28, 2021
@Walkyst Walkyst deleted the fix-404 branch June 29, 2021 06:59
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.

5 participants