Merge 98e034ff8c into 37cd7660ea

README.md

@@ -1869,9 +1869,6 @@ The following extractors use this feature:
 #### digitalconcerthall
 * `prefer_combined_hls`: Prefer extracting combined/pre-merged video and audio HLS formats. This will exclude 4K/HEVC video and lossless/FLAC audio formats, which are only available as split video/audio HLS formats
 
-#### sonylivseries
-* `sort_order`: Episode sort order for series extraction - one of `asc` (ascending, oldest first) or `desc` (descending, newest first). Default is `asc`
-
 **Note**: These options may be changed/removed in the future without concern for backward compatibility
 
 <!-- MANPAGE: MOVE "INSTALLATION" SECTION HERE -->

yt_dlp/extractor/ctvnews.py

@@ -1,27 +1,14 @@
-import json
 import re
-import urllib.parse
 
 from .common import InfoExtractor
-from .ninecninemedia import NineCNineMediaIE
+from ..utils import orderedSet
-from ..utils import extract_attributes, orderedSet
-from ..utils.traversal import find_element, traverse_obj
 
 
 class CTVNewsIE(InfoExtractor):
-    _BASE_REGEX = r'https?://(?:[^.]+\.)?ctvnews\.ca/'
+    _VALID_URL = r'https?://(?:.+?\.)?ctvnews\.ca/(?:video\?(?:clip|playlist|bin)Id=|.*?)(?P<id>[0-9.]+)'
-    _VIDEO_ID_RE = r'(?P<id>\d{5,})'
-    _PLAYLIST_ID_RE = r'(?P<id>\d\.\d{5,})'
-    _VALID_URL = [
-        rf'{_BASE_REGEX}video/c{_VIDEO_ID_RE}',
-        rf'{_BASE_REGEX}video(?:-gallery)?/?\?clipId={_VIDEO_ID_RE}',
-        rf'{_BASE_REGEX}video/?\?(?:playlist|bin)Id={_PLAYLIST_ID_RE}',
-        rf'{_BASE_REGEX}(?!video/)[^?#]*?{_PLAYLIST_ID_RE}/?(?:$|[?#])',
-        rf'{_BASE_REGEX}(?!video/)[^?#]+\?binId={_PLAYLIST_ID_RE}',
-    ]
     _TESTS = [{
         'url': 'http://www.ctvnews.ca/video?clipId=901995',
-        'md5': 'b608f466c7fa24b9666c6439d766ab7e',
+        'md5': '9b8624ba66351a23e0b6e1391971f9af',
         'info_dict': {
             'id': '901995',
             'ext': 'flv',
@@ -29,33 +16,6 @@ class CTVNewsIE(InfoExtractor):
             'description': 'md5:958dd3b4f5bbbf0ed4d045c790d89285',
             'timestamp': 1467286284,
             'upload_date': '20160630',
-            'categories': [],
-            'season_number': 0,
-            'season': 'Season 0',
-            'tags': [],
-            'series': 'CTV News National | Archive | Stories 2',
-            'season_id': '57981',
-            'thumbnail': r're:https?://.*\.jpg$',
-            'duration': 764.631,
-        },
-    }, {
-        'url': 'https://barrie.ctvnews.ca/video/c3030933-here_s-what_s-making-news-for-nov--15?binId=1272429',
-        'md5': '8b8c2b33c5c1803e3c26bc74ff8694d5',
-        'info_dict': {
-            'id': '3030933',
-            'ext': 'flv',
-            'title': 'Here’s what’s making news for Nov. 15',
-            'description': 'Here are the top stories we’re working on for CTV News at 11 for Nov. 15',
-            'thumbnail': 'http://images2.9c9media.com/image_asset/2021_2_22_a602e68e-1514-410e-a67a-e1f7cccbacab_png_2000x1125.jpg',
-            'season_id': '58104',
-            'season_number': 0,
-            'tags': [],
-            'season': 'Season 0',
-            'categories': [],
-            'series': 'CTV News Barrie',
-            'upload_date': '20241116',
-            'duration': 42.943,
-            'timestamp': 1731722452,
         },
     }, {
         'url': 'http://www.ctvnews.ca/video?playlistId=1.2966224',
@@ -71,72 +31,6 @@ class CTVNewsIE(InfoExtractor):
             'id': '1.2876780',
         },
         'playlist_mincount': 100,
-    }, {
-        'url': 'https://www.ctvnews.ca/it-s-been-23-years-since-toronto-called-in-the-army-after-a-major-snowstorm-1.5736957',
-        'info_dict':
-        {
-            'id': '1.5736957',
-        },
-        'playlist_mincount': 6,
-    }, {
-        'url': 'https://www.ctvnews.ca/business/respondents-to-bank-of-canada-questionnaire-largely-oppose-creating-a-digital-loonie-1.6665797',
-        'md5': '24bc4b88cdc17d8c3fc01dfc228ab72c',
-        'info_dict': {
-            'id': '2695026',
-            'ext': 'flv',
-            'season_id': '89852',
-            'series': 'From CTV News Channel',
-            'description': 'md5:796a985a23cacc7e1e2fafefd94afd0a',
-            'season': '2023',
-            'title': 'Bank of Canada asks public about digital currency',
-            'categories': [],
-            'tags': [],
-            'upload_date': '20230526',
-            'season_number': 2023,
-            'thumbnail': 'http://images2.9c9media.com/image_asset/2019_3_28_35f5afc3-10f6-4d92-b194-8b9a86f55c6a_png_1920x1080.jpg',
-            'timestamp': 1685105157,
-            'duration': 253.553,
-        },
-    }, {
-        'url': 'https://stox.ctvnews.ca/video-gallery?clipId=582589',
-        'md5': '135cc592df607d29dddc931f1b756ae2',
-        'info_dict': {
-            'id': '582589',
-            'ext': 'flv',
-            'categories': [],
-            'timestamp': 1427906183,
-            'season_number': 0,
-            'duration': 125.559,
-            'thumbnail': 'http://images2.9c9media.com/image_asset/2019_3_28_35f5afc3-10f6-4d92-b194-8b9a86f55c6a_png_1920x1080.jpg',
-            'series': 'CTV News Stox',
-            'description': 'CTV original footage of the rise and fall of the Berlin Wall.',
-            'title': 'Berlin Wall',
-            'season_id': '63817',
-            'season': 'Season 0',
-            'tags': [],
-            'upload_date': '20150401',
-        },
-    }, {
-        'url': 'https://ottawa.ctvnews.ca/features/regional-contact/regional-contact-archive?binId=1.1164587#3023759',
-        'md5': 'a14c0603557decc6531260791c23cc5e',
-        'info_dict': {
-            'id': '3023759',
-            'ext': 'flv',
-            'season_number': 2024,
-            'timestamp': 1731798000,
-            'season': '2024',
-            'episode': 'Episode 125',
-            'description': 'CTV News Ottawa at Six',
-            'duration': 2712.076,
-            'episode_number': 125,
-            'upload_date': '20241116',
-            'title': 'CTV News Ottawa at Six for Saturday, November 16, 2024',
-            'thumbnail': 'http://images2.9c9media.com/image_asset/2019_3_28_35f5afc3-10f6-4d92-b194-8b9a86f55c6a_png_1920x1080.jpg',
-            'categories': [],
-            'tags': [],
-            'series': 'CTV News Ottawa at Six',
-            'season_id': '92667',
-        },
     }, {
         'url': 'http://www.ctvnews.ca/1.810401',
         'only_matching': True,
@@ -148,35 +42,29 @@ class CTVNewsIE(InfoExtractor):
         'only_matching': True,
     }]
 
-    def _ninecninemedia_url_result(self, clip_id):
-        return self.url_result(f'9c9media:ctvnews_web:{clip_id}', NineCNineMediaIE, clip_id)
-
     def _real_extract(self, url):
         page_id = self._match_id(url)
 
-        if mobj := re.fullmatch(self._VIDEO_ID_RE, urllib.parse.urlparse(url).fragment):
+        def ninecninemedia_url_result(clip_id):
-            page_id = mobj.group('id')
+            return {
+                '_type': 'url_transparent',
+                'id': clip_id,
+                'url': f'9c9media:ctvnews_web:{clip_id}',
+                'ie_key': 'NineCNineMedia',
+            }
 
-        if re.fullmatch(self._VIDEO_ID_RE, page_id):
+        if page_id.isdigit():
-            return self._ninecninemedia_url_result(page_id)
+            return ninecninemedia_url_result(page_id)
-
+        else:
-        webpage = self._download_webpage(f'https://www.ctvnews.ca/{page_id}', page_id, query={
+            webpage = self._download_webpage(f'http://www.ctvnews.ca/{page_id}', page_id, query={
                 'ot': 'example.AjaxPageLayout.ot',
                 'maxItemsPerPage': 1000000,
             })
-        entries = [self._ninecninemedia_url_result(clip_id)
+            entries = [ninecninemedia_url_result(clip_id) for clip_id in orderedSet(
-                   for clip_id in orderedSet(re.findall(r'clip\.id\s*=\s*(\d+);', webpage))]
+                re.findall(r'clip\.id\s*=\s*(\d+);', webpage))]
             if not entries:
                 webpage = self._download_webpage(url, page_id)
                 if 'getAuthStates("' in webpage:
-                entries = [self._ninecninemedia_url_result(clip_id) for clip_id in
+                    entries = [ninecninemedia_url_result(clip_id) for clip_id in
                                self._search_regex(r'getAuthStates\("([\d+,]+)"', webpage, 'clip ids').split(',')]
-            else:
-                entries = [
-                    self._ninecninemedia_url_result(clip_id) for clip_id in
-                    traverse_obj(webpage, (
-                        {find_element(tag='jasper-player-container', html=True)},
-                        {extract_attributes}, 'axis-ids', {json.loads}, ..., 'axisId'))
-                ]
-
             return self.playlist_result(entries, page_id)

yt_dlp/extractor/sonyliv.py

@@ -199,9 +199,8 @@ class SonyLIVSeriesIE(InfoExtractor):
         },
     }]
     _API_BASE = 'https://apiv2.sonyliv.com/AGL'
-    _SORT_ORDERS = ('asc', 'desc')
 
-    def _entries(self, show_id, sort_order):
+    def _entries(self, show_id):
         headers = {
             'Accept': 'application/json, text/plain, */*',
             'Referer': 'https://www.sonyliv.com',
@@ -216,9 +215,6 @@ class SonyLIVSeriesIE(InfoExtractor):
                 'from': '0',
                 'to': '49',
             }), ('resultObj', 'containers', 0, 'containers', lambda _, v: int_or_none(v['id'])))
-
-        if sort_order == 'desc':
-            seasons = reversed(seasons)
         for season in seasons:
             season_id = str(season['id'])
             note = traverse_obj(season, ('metadata', 'title', {str})) or 'season'
@@ -230,7 +226,7 @@ class SonyLIVSeriesIE(InfoExtractor):
                         'from': str(cursor),
                         'to': str(cursor + 99),
                         'orderBy': 'episodeNumber',
-                        'sortOrder': sort_order,
+                        'sortOrder': 'asc',
                     }), ('resultObj', 'containers', 0, 'containers', lambda _, v: int_or_none(v['id'])))
                 if not episodes:
                     break
@@ -241,10 +237,4 @@ class SonyLIVSeriesIE(InfoExtractor):
 
     def _real_extract(self, url):
         show_id = self._match_id(url)
-
+        return self.playlist_result(self._entries(show_id), playlist_id=show_id)
-        sort_order = self._configuration_arg('sort_order', [self._SORT_ORDERS[0]])[0]
-        if sort_order not in self._SORT_ORDERS:
-            raise ValueError(
-                f'Invalid sort order "{sort_order}". Allowed values are: {", ".join(self._SORT_ORDERS)}')
-
-        return self.playlist_result(self._entries(show_id, sort_order), playlist_id=show_id)