-
Notifications
You must be signed in to change notification settings - Fork 368
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
[Broken] aenetworks - Partial fix with PR #228
Comments
It seems play.history.com is dealing with the same issue. |
Had to manually put your code change into mine, as your repo shows a DMCA takedown :-/ |
If you notice, the pull request is NOT mine. I merged it, and tested it, and it works some of the time. I was pointing out that it was only a partial fix. |
Looping back to this, I grabbed the code changes that were merged into ytdl-org and both of the links you provided appear to work without any issue. I'll file a PR for this. |
https://play.mylifetime.com/shows/married-at-first-sight/season-11/episode-14 works for you? I still get a 401 error. I have tried with two different provider logins. I can view it in the browser. |
I just did a PR for the fix this morning. Do you have the source code you can run? Otherwise you'll have to wait for the next release |
I merged it as soon as the PR showed up on youtube-dl. As I said, it is only a partial fix. Not all episodes detect properly. |
I tried the two that were listed and both worked. Are there other examples that don't work? I'd like to make sure it's a complete fix before merging it |
Definitely not working correctly for me on all aenetworks urls. The two referenced above, the first one works, but the second, I get a different error now. WHat MSO provider are you using? I have tried DTV and Charter_Direct. Its my understanding that Charter_Direct has been deprecated, but Spectrum isnt working yet.
|
Even without an MSO login it works:
|
Checklist
Verbose log
This URL works:
https://play.mylifetime.com/shows/married-at-first-sight/season-11/episode-17
This URL does not work:
https://play.mylifetime.com/shows/married-at-first-sight/season-11/episode-14
Description
I merged this pull request: ytdl-org/youtube-dl#26795
That works for some videos, but not all.
Thanks!
The text was updated successfully, but these errors were encountered: