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

Cinemassacre extractor stopped working #4008

Closed
marek-saji opened this issue Oct 23, 2014 · 1 comment
Closed

Cinemassacre extractor stopped working #4008

marek-saji opened this issue Oct 23, 2014 · 1 comment

Comments

@marek-saji
Copy link

@marek-saji marek-saji commented Oct 23, 2014

[debug] System config: []
[debug] User config: []mand to continue
[debug] Command-line args: ['--verbose', '--get-url', 'http://cinemassacre.com/2014/10/22/island-of-lost-souls-1932']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.10.18
[debug] Python version 2.7.6 - Linux-3.13.0-37-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] Proxy map: {}
ERROR: Can't extract embed url and video id; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
  File "/home/saji/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 524, in extract_info
    ie_result = ie.extract(url)
  File "/home/saji/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 193, in extract
    return self._real_extract(url)
  File "/home/saji/local/bin/youtube-dl/youtube_dl/extractor/cinemassacre.py", line 47, in _real_extract
    raise ExtractorError('Can\'t extract embed url and video id')
ExtractorError: Can't extract embed url and video id; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

First thing is that playerdata_url changed and now includes extra query parameters. But even after changing regexp in cinemassacre extractor to:

        mobj = re.search(r'src="(?P<embed_url>http://player\.screenwavemedia\.com/play/[a-zA-Z]+\.php\?(?:[^"]*&)?id=(?:Cinemassacre-)?(?P<video_id>.+?))"', webpage)

I still get errors. Seems that playerdata file format changed quite a bit.

[debug] User config: []
[debug] Command-line args: ['--verbose', '--get-url', 'http://cinemassacre.com/2014/10/22/island-of-lost-souls-1932/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.10.18
[debug] Git HEAD: 6b44555
[debug] Python version 2.7.6 - Linux-3.13.0-37-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] Proxy map: {}
WARNING: unable to extract thumbnail; please report this issue on http://yt-dl.org/bug
ERROR: Unable to extract sd_file; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
  File "/home/saji/local/src/youtube-dl/youtube_dl/YoutubeDL.py", line 524, in extract_info
    ie_result = ie.extract(url)
  File "/home/saji/local/src/youtube-dl/youtube_dl/extractor/common.py", line 193, in extract
    return self._real_extract(url)
  File "/home/saji/local/src/youtube-dl/youtube_dl/extractor/cinemassacre.py", line 60, in _real_extract
    sd_url = self._search_regex(r'file: \'([^\']+)\', label: \'SD\'', playerdata, 'sd_file')
  File "/home/saji/local/src/youtube-dl/youtube_dl/extractor/common.py", line 424, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract sd_file; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
@dstftw dstftw closed this in 8f3b539 Oct 23, 2014
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Oct 23, 2014

Fixed and will be available in the next release. Thanks for the report.

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
2 participants
You can’t perform that action at this time.