yt_dlp/extractor/duoplay.py

@@ -5,16 +5,16 @@ from ..utils import (
     get_element_text_and_html_by_tag,
     int_or_none,
     join_nonempty,
-    parse_qs,
     str_or_none,
     try_call,
     unified_timestamp,
+    update_url_query,
 )
-from ..utils.traversal import traverse_obj, value
+from ..utils.traversal import traverse_obj
 
 
 class DuoplayIE(InfoExtractor):
-    _VALID_URL = r'https?://duoplay\.ee/(?P<id>\d+)(?:[/?#]|$)'
+    _VALID_URL = r'https?://duoplay\.ee/(?P<id>\d+)(?:/[\w-]+/?(?:\?(?:[^#]+&))?ep=(?P<ep>\d+))?'
     _TESTS = [{
         'note': 'Siberi võmm S02E12',
         'url': 'https://duoplay.ee/4312/siberi-vomm?ep=24',
@@ -35,16 +35,15 @@ class DuoplayIE(InfoExtractor):
             'episode_number': 12,
             'episode_id': '24',
         },
-        'skip': 'No video found',
     }, {
         'note': 'Empty title',
         'url': 'https://duoplay.ee/17/uhikarotid?ep=14',
-        'md5': 'cba9f5dabf2582b224d80ac44fb80e47',
+        'md5': '6aca68be71112314738dd17cced7f8bf',
         'info_dict': {
             'id': '17_14',
             'ext': 'mp4',
-            'title': 'Episode 14',
-            'thumbnail': r're:https?://.+\.jpg',
+            'title': 'Ühikarotid',
+            'thumbnail': r're:https://.+\.jpg(?:\?c=\d+)?$',
             'description': 'md5:4719b418e058c209def41d48b601276e',
             'upload_date': '20100916',
             'timestamp': 1284661800,
@@ -54,8 +53,6 @@ class DuoplayIE(InfoExtractor):
             'season_number': 2,
             'episode_id': '14',
             'release_year': 2010,
-            'episode': 'Episode 14',
-            'episode_number': 14,
         },
     }, {
         'note': 'Movie without expiry',
@@ -72,17 +69,17 @@ class DuoplayIE(InfoExtractor):
             'timestamp': 1671054000,
             'release_year': 2018,
         },
-        'skip': 'No video found',
     }, {
         'note': 'Episode url without show name',
         'url': 'https://duoplay.ee/9644?ep=185',
         'md5': '63f324b4fe2dbd8194dca16a6d52184a',
         'info_dict': {
-            'id': '9644_185',
+            'id': '9644',
             'ext': 'mp4',
-            'title': 'Episode 185',
-            'thumbnail': r're:https?://.+\.jpg',
+            'title': 'Episode 63',
+            'thumbnail': r're:https?://.+\.jpg(?:%3Fc%3D\d+)?$',
             'description': 'md5:ed25ba4e9e5d54bc291a4a0cdd241467',
+            'cast': 'count:1',
             'upload_date': '20241120',
             'timestamp': 1732077000,
             'episode': 'Episode 63',
@@ -96,8 +93,7 @@ class DuoplayIE(InfoExtractor):
     }]
 
     def _real_extract(self, url):
-        telecast_id = self._match_id(url)
-        episode = traverse_obj(parse_qs(url), ('ep', 0, {int_or_none}, {str_or_none}))
+        telecast_id, episode = self._match_valid_url(url).group('id', 'ep')
         video_id = join_nonempty(telecast_id, episode, delim='_')
         webpage = self._download_webpage(url, video_id)
         video_player = try_call(lambda: extract_attributes(
@@ -112,26 +108,27 @@ class DuoplayIE(InfoExtractor):
                 'Accept': 'application/json',
                 'X-Original-URI': manifest_url,
             })['session']
+        manifest_url = update_url_query(manifest_url, {'s': session_token})
 
         episode_attr = self._parse_json(video_player.get(':episode') or '', video_id, fatal=False) or {}
 
         return {
             'id': video_id,
-            'formats': self._extract_m3u8_formats(manifest_url, video_id, 'mp4', query={'s': session_token}),
+            'formats': self._extract_m3u8_formats(manifest_url, video_id, 'mp4'),
             **traverse_obj(episode_attr, {
-                'title': ('title', {str}),
-                'description': ('synopsis', {str}),
+                'title': 'title',
+                'description': 'synopsis',
                 'thumbnail': ('images', 'original'),
                 'timestamp': ('airtime', {lambda x: unified_timestamp(x + ' +0200')}),
-                'cast': ('cast', filter, {lambda x: x.split(', ')}),
+                'cast': ('cast', {lambda x: x.split(', ')}),
                 'release_year': ('year', {int_or_none}),
             }),
             **(traverse_obj(episode_attr, {
-                'title': (None, (('subtitle', {str}, filter), {value(f'Episode {episode}' if episode else None)})),
-                'series': ('title', {str}),
+                'title': (None, ('subtitle', ('episode_nr', {lambda x: f'Episode {x}' if x else None}))),
+                'series': 'title',
                 'series_id': ('telecast_id', {str_or_none}),
                 'season_number': ('season_id', {int_or_none}),
-                'episode': ('subtitle', {str}, filter),
+                'episode': 'subtitle',
                 'episode_number': ('episode_nr', {int_or_none}),
                 'episode_id': ('episode_id', {str_or_none}),
             }, get_all=False) if episode_attr.get('category') != 'movies' else {}),