aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNaglis Jonaitis <njonaitis@gmail.com>2014-11-03 23:12:24 +0200
committerNaglis Jonaitis <njonaitis@gmail.com>2014-11-03 23:12:24 +0200
commitb5af6fcdaddeade533b82708a137f4e5bed744df (patch)
treed4c7b326d408a33f26f914d1ddb0597ff730ccec
parent278143df5b026f76e52bad8500a32780b52a1e35 (diff)
[brightcove] Make _VALID_URL less greedy and check for empty URLs (#3541)
-rw-r--r--youtube_dl/extractor/brightcove.py4
1 files changed, 3 insertions, 1 deletions
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<query>.*)'
+ _VALID_URL = r'https?://.*brightcove\.com/(services|viewer).*?\?(?P<query>.*)'
_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