Skip to content
This repository has been archived by the owner on Sep 28, 2020. It is now read-only.

Change player javascript path #351

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aleksey-sh
Copy link

I am not sure whether this is a YouTube change that caused the code not working or its something specific to my profile that is different (if applicable). I haven't been able to dig thru the code to understand it but my player js path looks a bit different. The js variable has this value - "/yts/jsbin/player_ias-vflx77j21/en_US/base.js".

Could someone with better understanding of youtube working take a look at this change and see if it makes sense?

@MilanDierick
Copy link

@aleksey-sh I don't really know how YouTube works, but I applied your change and successfully grabbed the title of a video, going to try and download it now...

@aleksey-sh
Copy link
Author

I actually noticed that there are several issues (and pull requests) that are related to this fix. In issue #341 (Player JSON changed, cannot decrypt) @s19k15 refers to the same change and adds another place that may be related.

@MilanDierick
Copy link

MilanDierick commented Apr 2, 2019

Something very strange is going on... I applied your fix, and I can download a couple of videos fine, but then all of the sudden, without me changing any part of my code, I get the error: "Could not parse the Youtube page for URL http://youtube.com/watch?v=_3xvt-apIic" .

EDIT:
Did some more reading, and it would seem that it's still not possible to download most music videos. The other thing I've noticed that some videos don't have any audio...

@aleksey-sh
Copy link
Author

aleksey-sh commented Apr 2, 2019

@MilanDierick

Funny thing. I've got similar error few times with my code as well. Rerunning fixed it. I am assuming some requests do not go thru as expected and return unexpected response. Did not get a chance to debug it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants