summary refs log tree commit diff
diff options
context:
space:
mode:
authorPhilipp Hagemeister <phihag@phihag.de>2013-12-09 19:57:00 +0100
committerPhilipp Hagemeister <phihag@phihag.de>2013-12-09 19:57:00 +0100
commit4ff50ef84607ee60ff813f7d7f3d35c8b497bf07 (patch)
tree8cbb032caaf9aa875dbf0143bcfe962dedcf59ef
parentcaefb1de877e163fa3ece44757cb2fae6adf47e4 (diff)
downloadyoutube-dl-4ff50ef84607ee60ff813f7d7f3d35c8b497bf07.tar.gz
youtube-dl-4ff50ef84607ee60ff813f7d7f3d35c8b497bf07.tar.xz
youtube-dl-4ff50ef84607ee60ff813f7d7f3d35c8b497bf07.zip
[soundcloud] Do not match sets (Fixes #1930)
-rw-r--r--test/test_all_urls.py3
-rw-r--r--youtube_dl/extractor/soundcloud.py3
2 files changed, 5 insertions, 1 deletions
diff --git a/test/test_all_urls.py b/test/test_all_urls.py
index 6b9764c67..e9458b2e3 100644
--- a/test/test_all_urls.py
+++ b/test/test_all_urls.py
@@ -110,6 +110,9 @@ class TestAllURLsMatching(unittest.TestCase):
         self.assertMatch('http://vimeo.com/channels/tributes', ['vimeo:channel'])
         self.assertMatch('http://vimeo.com/user7108434', ['vimeo:user'])
 
+    # https://github.com/rg3/youtube-dl/issues/1930
+    def test_soundcloud_not_matching_sets(self):
+        self.assertMatch('http://soundcloud.com/floex/sets/gone-ep', ['soundcloud:set'])
 
 if __name__ == '__main__':
     unittest.main()
diff --git a/youtube_dl/extractor/soundcloud.py b/youtube_dl/extractor/soundcloud.py
index 0571b36ac..5c026c0b8 100644
--- a/youtube_dl/extractor/soundcloud.py
+++ b/youtube_dl/extractor/soundcloud.py
@@ -25,7 +25,8 @@ class SoundcloudIE(InfoExtractor):
 
     _VALID_URL = r'''^(?:https?://)?
                     (?:(?:(?:www\.)?soundcloud\.com/
-                            (?P<uploader>[\w\d-]+)/(?P<title>[\w\d-]+)/?
+                            (?P<uploader>[\w\d-]+)/
+                            (?!sets/)(?P<title>[\w\d-]+)/?
                             (?P<token>[^?]+?)?(?:[?].*)?$)
                        |(?:api\.soundcloud\.com/tracks/(?P<track_id>\d+))
                        |(?P<widget>w\.soundcloud\.com/player/?.*?url=.*)