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

Decryption Key Length 82 changed #1539

Closed
cujo30227 opened this issue Oct 1, 2013 · 4 comments
Closed

Decryption Key Length 82 changed #1539

cujo30227 opened this issue Oct 1, 2013 · 4 comments

Comments

@cujo30227
Copy link

@cujo30227 cujo30227 commented Oct 1, 2013

New youtube static decryption code for len(s) == 82:
return s[12] + s[79:12:-1] + s[80] + s[11::-1]

@phihag
Copy link
Contributor

@phihag phihag commented Oct 1, 2013

Can you provide an example URL that's affected by this problem?

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Oct 2, 2013

We should really not be using lengths but js player version names.

@cujo30227
Copy link
Author

@cujo30227 cujo30227 commented Oct 2, 2013

Run PIb6AZdTr-A with --youtube-print-sig-code option to get signature decryption code. I noticed it's different to the current 82 length code. I'm also using automatic extraction as first option, but I thought we wanted to keep the static keys up to date as fallback option..

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Oct 2, 2013

@FiloSottile we currently use the player version with the automatic signature decryption, we fallback to the old system if it founds an error.

We could remove the static signature decryption for normal videos (but keep it for age-gate videos). If we fallback to it, the algorithms will probably be outdated, we would then report the problem right there, instead of getting a 403 error during the download.

@phihag phihag closed this in ca40186 Oct 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.