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.

Sign up for GitHub

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

Jump to bottom

[Youku] fix extractor to use latest ccode #28460

Closed
5 tasks done
peroperosauce opened this issue Mar 16, 2021 · 1 comment
Closed
5 tasks done

[Youku] fix extractor to use latest ccode #28460

peroperosauce opened this issue Mar 16, 2021 · 1 comment

Comments

@peroperosauce
Copy link

Checklist

  • I'm reporting a broken site support
  • I've verified that I'm running youtube-dl version 2021.03.14
  • I've checked that all provided URLs are alive and playable in a browser
  • I've checked that all URLs and arguments with special characters are properly quoted or escaped
  • I've searched the bugtracker for similar issues including closed ones

Verbose log

ERROR: Youku server reported error -6004: 客户端无权播放,201; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type youtube-dl -U to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

In youku.py, as of today 2021.03.16, the currently hard-coded ccode 0590 :
https://github.com/ytdl-org/youtube-dl/blob/master/youtube_dl/extractor/youku.py#L157

does not work anymore and leads to the following error when trying to download any youku video :
ERROR: Youku server reported error -6004: 客户端无权播放,201

The new ccode to use since march 2021 is 0532.
Can you please update youky.py in order to replace 0590 by 0532 ?

I asked the question in #28447 but maybe an issue "Broken site support" is more suitable ?
Thanks
Open
5 tasks
@W-Leviosa
Copy link

Hello, I am using mac OS, can anyone help with where should I change this code? I don't know how to find the youku.py file.

@RomanEmelyanov RomanEmelyanov mentioned this issue Apr 3, 2021
Merged
11 tasks
pull bot pushed a commit to JonasGessner/youtube-dl that referenced this issue Apr 4, 2021
@RomanEmelyanov @pull
[youku] Update ccode(closes ytdl-org#17852, closes ytdl-org#28447, cl…
5cac2ef 
…oses ytdl-org#28460) (ytdl-org#28648)
github-actions bot added a commit to hellopony/youtube-dl that referenced this issue Apr 4, 2021
@github-actions
[bot] AutoMerging: merge all upstream's changes:
b4f69f4 
* https://github.com/ytdl-org/youtube-dl:
  [compat] add compat_SimpleCookie
  [extractor/common] keep support for non standard JSON-LD VideoObject author values
  [vimeo] improve extraction(closes ytdl-org#28591)
  [extractor/common] fix JSON-LD VideoObject author extraction
  [youku] Update ccode(closes ytdl-org#17852, closes ytdl-org#28447, closes ytdl-org#28460) (ytdl-org#28648)
@LennaHammer LennaHammer mentioned this issue Aug 19, 2022
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@peroperosauce @W-Leviosa