From a8df272cabe79552524759c7a1231c420603c9a9 Mon Sep 17 00:00:00 2001
From: John Hawkinson <jhawk@mit.edu>
Date: Sat, 8 Apr 2017 05:33:25 -0400
Subject: [PATCH] [Kaltura] comment; anchor RE to 'iframe'

Remove the example of an iframe spec from the spec page, and instead
link to it. Do so at the top of the method as it documents the
prviously two re.search()s as well.

Limit the iframe search to urls that actually appear in an <iframe
src="...  rather than inside any quoted string.
---
 youtube_dl/extractor/kaltura.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/youtube_dl/extractor/kaltura.py b/youtube_dl/extractor/kaltura.py
index 6e992ee4b..c6efdea93 100644
--- a/youtube_dl/extractor/kaltura.py
+++ b/youtube_dl/extractor/kaltura.py
@@ -109,6 +109,7 @@ class KalturaIE(InfoExtractor):
     @staticmethod
     def _extract_url(webpage):
         mobj = (
+            # Embed codes: https://knowledge.kaltura.com/embedding-kaltura-media-players-your-site
             re.search(
                 r"""(?xs)
                     kWidget\.(?:thumb)?[Ee]mbed\(
@@ -130,9 +131,8 @@ class KalturaIE(InfoExtractor):
                     (?P<q3>["\'])(?P<id>(?:(?!(?P=q3)).)+)(?P=q3)
                 ''', webpage) or
             re.search(
-                # <iframe src="http://www.kaltura.com/p/{PARTNER_ID}/sp/{PARTNER_ID}00/embedIframeJs/uiconf_id/{UICONF_ID}/partner_id/{PARTNER_ID}?iframeembed=true&playerId={UNIQUE_OBJ_ID}&entry_id={ENTRY_ID}" width="400" height="330" allowfullscreen webkitallowfullscreen mozAllowFullScreen frameborder="0"></iframe>
                 r'''(?xs)
-                    (?P<q1>["\'])
+                    <iframe[^>]+src=(?P<q1>["\'])
                       (?:https?:)?//(?:www\.)?kaltura\.com/p/(?P<partner_id>\d+)/
                       (?:(?!(?P=q1)).)*
                       [\?&]entry_id=(?P<id>(?:(?!(?P=q1))[^&])+)