From b5af6fcdaddeade533b82708a137f4e5bed744df Mon Sep 17 00:00:00 2001 From: Naglis Jonaitis Date: Mon, 3 Nov 2014 23:12:24 +0200 Subject: [PATCH] [brightcove] Make _VALID_URL less greedy and check for empty URLs (#3541) --- youtube_dl/extractor/brightcove.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/youtube_dl/extractor/brightcove.py b/youtube_dl/extractor/brightcove.py index ad22cbafd..cc94e596c 100644 --- a/youtube_dl/extractor/brightcove.py +++ b/youtube_dl/extractor/brightcove.py @@ -23,7 +23,7 @@ from ..utils import ( class BrightcoveIE(InfoExtractor): - _VALID_URL = r'https?://.*brightcove\.com/(services|viewer).*\?(?P.*)' + _VALID_URL = r'https?://.*brightcove\.com/(services|viewer).*?\?(?P.*)' _FEDERATED_URL_TEMPLATE = 'http://c.brightcove.com/services/viewer/htmlFederated?%s' _TESTS = [ @@ -260,6 +260,8 @@ class BrightcoveIE(InfoExtractor): formats = [] for rend in renditions: url = rend['defaultURL'] + if not url: + continue if rend['remote']: # This type of renditions are served through akamaihd.net, # but they don't use f4m manifests -- 2.22.2