debian/control: Mark compliance with Debian policy 4.1.5.

[youtubedl] / youtube_dl / extractor / sohu.py

diff --git a/youtube_dl/extractor/sohu.py b/youtube_dl/extractor/sohu.py
index ba2d5e19bc0d1de322b4b12ed5b8c0dc31157f7f..a62ed84f1e34b75a721914d68a101ad2a0bb8983 100644 (file)
--- a/youtube_dl/extractor/sohu.py
+++ b/youtube_dl/extractor/sohu.py
@@ -1,4 +1,4 @@
-# encoding: utf-8
+# coding: utf-8

 from __future__ import unicode_literals
 
 import re
 from __future__ import unicode_literals
 
 import re


@@ -6,21 +6,22 @@ import re
 from .common import InfoExtractor
 from ..compat import (
     compat_str,
 from .common import InfoExtractor
 from ..compat import (
     compat_str,

-    compat_urllib_request,
-    compat_urllib_parse,
+    compat_urllib_parse_urlencode,

 )
 from ..utils import (
     ExtractorError,
 )
 from ..utils import (
     ExtractorError,

+    int_or_none,
+    try_get,

 )
 
 
 class SohuIE(InfoExtractor):
     _VALID_URL = r'https?://(?P<mytv>my\.)?tv\.sohu\.com/.+?/(?(mytv)|n)(?P<id>\d+)\.shtml.*?'
 
 )
 
 
 class SohuIE(InfoExtractor):
     _VALID_URL = r'https?://(?P<mytv>my\.)?tv\.sohu\.com/.+?/(?(mytv)|n)(?P<id>\d+)\.shtml.*?'
 

+    # Sohu videos give different MD5 sums on Travis CI and my machine

     _TESTS = [{
         'note': 'This video is available only in Mainland China',
         'url': 'http://tv.sohu.com/20130724/n382479172.shtml#super',
     _TESTS = [{
         'note': 'This video is available only in Mainland China',
         'url': 'http://tv.sohu.com/20130724/n382479172.shtml#super',

-        'md5': '29175c8cadd8b5cc4055001e85d6b372',

         'info_dict': {
             'id': '382479172',
             'ext': 'mp4',
         'info_dict': {
             'id': '382479172',
             'ext': 'mp4',


@@ -29,7 +30,6 @@ class SohuIE(InfoExtractor):
         'skip': 'On available in China',
     }, {
         'url': 'http://tv.sohu.com/20150305/n409385080.shtml',
         'skip': 'On available in China',
     }, {
         'url': 'http://tv.sohu.com/20150305/n409385080.shtml',

-        'md5': '699060e75cf58858dd47fb9c03c42cfb',

         'info_dict': {
             'id': '409385080',
             'ext': 'mp4',
         'info_dict': {
             'id': '409385080',
             'ext': 'mp4',


@@ -37,7 +37,6 @@ class SohuIE(InfoExtractor):
         }
     }, {
         'url': 'http://my.tv.sohu.com/us/232799889/78693464.shtml',
         }
     }, {
         'url': 'http://my.tv.sohu.com/us/232799889/78693464.shtml',

-        'md5': '9bf34be48f2f4dadcb226c74127e203c',

         'info_dict': {
             'id': '78693464',
             'ext': 'mp4',
         'info_dict': {
             'id': '78693464',
             'ext': 'mp4',


@@ -51,7 +50,6 @@ class SohuIE(InfoExtractor):
             'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
         },
         'playlist': [{
             'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
         },
         'playlist': [{

-            'md5': 'bdbfb8f39924725e6589c146bc1883ad',

             'info_dict': {
                 'id': '78910339_part1',
                 'ext': 'mp4',
             'info_dict': {
                 'id': '78910339_part1',
                 'ext': 'mp4',


@@ -59,7 +57,6 @@ class SohuIE(InfoExtractor):
                 'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
             }
         }, {
                 'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
             }
         }, {

-            'md5': '3e1f46aaeb95354fd10e7fca9fc1804e',

             'info_dict': {
                 'id': '78910339_part2',
                 'ext': 'mp4',
             'info_dict': {
                 'id': '78910339_part2',
                 'ext': 'mp4',


@@ -67,7 +64,6 @@ class SohuIE(InfoExtractor):
                 'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
             }
         }, {
                 'title': '【神探苍实战秘籍】第13期 战争之影 赫卡里姆',
             }
         }, {

-            'md5': '8407e634175fdac706766481b9443450',

             'info_dict': {
                 'id': '78910339_part3',
                 'ext': 'mp4',
             'info_dict': {
                 'id': '78910339_part3',
                 'ext': 'mp4',


@@ -96,15 +92,10 @@ class SohuIE(InfoExtractor):
             else:
                 base_data_url = 'http://hot.vrs.sohu.com/vrs_flash.action?vid='
 
             else:
                 base_data_url = 'http://hot.vrs.sohu.com/vrs_flash.action?vid='
 

-            req = compat_urllib_request.Request(base_data_url + vid_id)
-
-            cn_verification_proxy = self._downloader.params.get('cn_verification_proxy')
-            if cn_verification_proxy:
-                req.add_header('Ytdl-request-proxy', cn_verification_proxy)
-

             return self._download_json(
             return self._download_json(

-                req, video_id,
-                'Downloading JSON data for %s' % vid_id)
+                base_data_url + vid_id, video_id,
+                'Downloading JSON data for %s' % vid_id,
+                headers=self.geo_verification_headers())

 
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')
 
         mobj = re.match(self._VALID_URL, url)
         video_id = mobj.group('id')


@@ -121,12 +112,11 @@ class SohuIE(InfoExtractor):
         if vid_data['play'] != 1:
             if vid_data.get('status') == 12:
                 raise ExtractorError(
         if vid_data['play'] != 1:
             if vid_data.get('status') == 12:
                 raise ExtractorError(

-                    'Sohu said: There\'s something wrong in the video.',
+                    '%s said: There\'s something wrong in the video.' % self.IE_NAME,

                     expected=True)
             else:
                     expected=True)
             else:

-                raise ExtractorError(
-                    'Sohu said: The video is only licensed to users in Mainland China.',
-                    expected=True)
+                self.raise_geo_restricted(
+                    '%s said: The video is only licensed to users in Mainland China.' % self.IE_NAME)

 
         formats_json = {}
         for format_id in ('nor', 'high', 'super', 'ori', 'h2644k', 'h2654k'):
 
         formats_json = {}
         for format_id in ('nor', 'high', 'super', 'ori', 'h2644k', 'h2654k'):


@@ -158,6 +148,7 @@ class SohuIE(InfoExtractor):
                         'file': clips_url[i],
                         'new': su[i],
                         'prod': 'flash',
                         'file': clips_url[i],
                         'new': su[i],
                         'prod': 'flash',

+                        'rb': 1,

                     }
 
                     if cdnId is not None:
                     }
 
                     if cdnId is not None:


@@ -169,7 +160,7 @@ class SohuIE(InfoExtractor):
                     if retries > 0:
                         download_note += ' (retry #%d)' % retries
                     part_info = self._parse_json(self._download_webpage(
                     if retries > 0:
                         download_note += ' (retry #%d)' % retries
                     part_info = self._parse_json(self._download_webpage(

-                        'http://%s/?%s' % (allot, compat_urllib_parse.urlencode(params)),
+                        'http://%s/?%s' % (allot, compat_urllib_parse_urlencode(params)),

                         video_id, download_note), video_id)
 
                     video_url = part_info['url']
                         video_id, download_note), video_id)
 
                     video_url = part_info['url']


@@ -182,10 +173,11 @@ class SohuIE(InfoExtractor):
                 formats.append({
                     'url': video_url,
                     'format_id': format_id,
                 formats.append({
                     'url': video_url,
                     'format_id': format_id,

-                    'filesize': data['clipsBytes'][i],
-                    'width': data['width'],
-                    'height': data['height'],
-                    'fps': data['fps'],
+                    'filesize': int_or_none(
+                        try_get(data, lambda x: x['clipsBytes'][i])),
+                    'width': int_or_none(data.get('width')),
+                    'height': int_or_none(data.get('height')),
+                    'fps': int_or_none(data.get('fps')),

                 })
             self._sort_formats(formats)
 
                 })
             self._sort_formats(formats)