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 Youtube encrypted sigs. #4706

Merged
merged 1 commit into from Jan 15, 2015
Merged

Fix Youtube encrypted sigs. #4706

merged 1 commit into from Jan 15, 2015

Conversation

pkulak
Copy link
Contributor

@pkulak pkulak commented Jan 15, 2015

Looks like Google switched to a new JS compiler that includes dollar signs in function names.

@pkulak
Copy link
Contributor Author

pkulak commented Jan 15, 2015

Sorry, worst commit message ever. Not encrypted videos, encrypted sigs.

@dstftw
Copy link
Collaborator

dstftw commented Jan 15, 2015

Can you name an example URL?

@dstftw dstftw changed the title Fix Youtube encrypted videos. Fix Youtube encrypted sigs. Jan 15, 2015
@pkulak
Copy link
Contributor Author

pkulak commented Jan 15, 2015

Yeah. All videos with encrypted sigs are failing for me. For example:

nfWlot6h_JM
On Jan 15, 2015 10:46 AM, "Sergey M." notifications@github.com wrote:

Can you name an example URL?


Reply to this email directly or view it on GitHub
#4706 (comment).

dstftw added a commit that referenced this pull request Jan 15, 2015
Fix Youtube encrypted sigs.
@dstftw dstftw merged commit 9fa6ea2 into ytdl-org:master Jan 15, 2015
dstftw added a commit that referenced this pull request Jan 15, 2015
@dstftw
Copy link
Collaborator

dstftw commented Jan 15, 2015

Thanks.

@pkulak
Copy link
Contributor Author

pkulak commented Jan 15, 2015

No prob!
On Jan 15, 2015 11:26 AM, "Sergey M." notifications@github.com wrote:

Thanks.


Reply to this email directly or view it on GitHub
#4706 (comment).

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.

None yet

2 participants