debian/control: Mark compliance with policy 4.1.1. No further changes needed.

[youtubedl] / youtube_dl / extractor / ooyala.py

diff --git a/youtube_dl/extractor/ooyala.py b/youtube_dl/extractor/ooyala.py
index 84be2b1e3fe9fec47e915c13b14a8f23c0383232..52580baed3c49242be55529f8c75f5d110ff41f7 100644 (file)
--- a/youtube_dl/extractor/ooyala.py
+++ b/youtube_dl/extractor/ooyala.py
@@ -3,12 +3,14 @@ import re
 import base64
 
 from .common import InfoExtractor
 import base64
 
 from .common import InfoExtractor

+from ..compat import compat_str

 from ..utils import (
 from ..utils import (

-    int_or_none,
-    float_or_none,
+    determine_ext,

     ExtractorError,
     ExtractorError,

+    float_or_none,
+    int_or_none,
+    try_get,

     unsmuggle_url,
     unsmuggle_url,

-    determine_ext,

 )
 from ..compat import compat_urllib_parse_urlencode
 
 )
 from ..compat import compat_urllib_parse_urlencode
 


@@ -39,13 +41,15 @@ class OoyalaBaseIE(InfoExtractor):
         formats = []
         if cur_auth_data['authorized']:
             for stream in cur_auth_data['streams']:
         formats = []
         if cur_auth_data['authorized']:
             for stream in cur_auth_data['streams']:

-                s_url = base64.b64decode(
-                    stream['url']['data'].encode('ascii')).decode('utf-8')
-                if s_url in urls:
+                url_data = try_get(stream, lambda x: x['url']['data'], compat_str)
+                if not url_data:
+                    continue
+                s_url = base64.b64decode(url_data.encode('ascii')).decode('utf-8')
+                if not s_url or s_url in urls:

                     continue
                 urls.append(s_url)
                 ext = determine_ext(s_url, None)
                     continue
                 urls.append(s_url)
                 ext = determine_ext(s_url, None)

-                delivery_type = stream['delivery_type']
+                delivery_type = stream.get('delivery_type')

                 if delivery_type == 'hls' or ext == 'm3u8':
                     formats.extend(self._extract_m3u8_formats(
                         re.sub(r'/ip(?:ad|hone)/', '/all/', s_url), embed_code, 'mp4', 'm3u8_native',
                 if delivery_type == 'hls' or ext == 'm3u8':
                     formats.extend(self._extract_m3u8_formats(
                         re.sub(r'/ip(?:ad|hone)/', '/all/', s_url), embed_code, 'mp4', 'm3u8_native',


@@ -65,7 +69,7 @@ class OoyalaBaseIE(InfoExtractor):
                 else:
                     formats.append({
                         'url': s_url,
                 else:
                     formats.append({
                         'url': s_url,

-                        'ext': ext or stream.get('delivery_type'),
+                        'ext': ext or delivery_type,

                         'vcodec': stream.get('video_codec'),
                         'format_id': delivery_type,
                         'width': int_or_none(stream.get('width')),
                         'vcodec': stream.get('video_codec'),
                         'format_id': delivery_type,
                         'width': int_or_none(stream.get('width')),


@@ -136,6 +140,11 @@ class OoyalaIE(OoyalaBaseIE):
                 'title': 'Divide Tool Path.mp4',
                 'duration': 204.405,
             }
                 'title': 'Divide Tool Path.mp4',
                 'duration': 204.405,
             }

+        },
+        {
+            # empty stream['url']['data']
+            'url': 'http://player.ooyala.com/player.js?embedCode=w2bnZtYjE6axZ_dw1Cd0hQtXd_ige2Is',
+            'only_matching': True,

         }
     ]
 
         }
     ]