-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
[toutv] fixes #16398 - accounts don't work after site redesign. #18700
Conversation
Hey kudos for the fix ! I am new as of today to youtube-dl, but it worked flawlessly after doing a checkout of your toutv_account branch. |
Do you need anymore changes? Thanks |
I tried running this branch and while it seems I can login, there is an error afterwards.
|
Weird, i just tried it and it worked. Looked like the server send a timeout error. Maybe give it another go. I notice tou.tv seems to have server issues recently. Bonne chance! |
Franchement, j'aurais pu essayer deux fois... ça fonctionne bien, merci! =) |
@dstftw Is there any reason why this PR in not getting merge? |
Sent my account info to @remitamine, hope this will help. |
Please follow the guide below
x
into all the boxes [ ] relevant to your pull request (like that [x])Before submitting a pull request make sure you have:
In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:
What is the purpose of your pull request?
Description of your pull request and other information
fixes #16398
Tou.tv was redesign with react and the login url was moved into JS. The code was not able to find the clientID needed.
My fix browses through the app.js file to extract the clientID needed for the login form.