From 1b4110c8deabf91f8f1010c084a9548ac6b21795 Mon Sep 17 00:00:00 2001 From: Simone Dotto Date: Sun, 15 Jun 2025 16:16:48 +0200 Subject: [PATCH] [ie/IlPost] Fix extractor - merge review pt2 --- yt_dlp/extractor/ilpost.py | 103 ++++++++++++++++++++----------------- 1 file changed, 57 insertions(+), 46 deletions(-) diff --git a/yt_dlp/extractor/ilpost.py b/yt_dlp/extractor/ilpost.py index 5f7a59fc0..66b46c027 100644 --- a/yt_dlp/extractor/ilpost.py +++ b/yt_dlp/extractor/ilpost.py @@ -1,3 +1,5 @@ +import itertools + from .common import InfoExtractor from ..utils import ( clean_html, @@ -11,25 +13,24 @@ class IlPostIE(InfoExtractor): _VALID_URL = r'https?://(?:www\.)?ilpost\.it/podcasts/[^/?#]+/(?P[^/?#]+)' - _TESTS = [ - { - 'url': 'https://www.ilpost.it/podcasts/timbuctu/ep-323-lanno-record-della-pena-di-morte/', - 'md5': '55d88cc23bcab991639ebcbf1b4c0aa1', - 'info_dict': { - 'id': '3326553', - 'ext': 'mp3', - 'display_id': 'ep-323-lanno-record-della-pena-di-morte', - 'title': 'Ep. 323 – L’anno record della pena di morte', - 'url': 'https://static-prod.cdnilpost.com/wp-content/uploads/2025/05/25/1748196012-timbuctu_250526_v1_-16lufs.mp3', - 'timestamp': 1748235641, - 'upload_date': '20250526', - 'description': 'md5:331514a14779fab06e902160ec8c89ba', - 'duration': 751, - 'availability': 'public', - 'series_id': '233679', - 'thumbnail': 'https://www.ilpost.it/wp-content/uploads/2023/05/19/1684536738-copertina500x500.jpg', - }, + _TESTS = [{ + 'url': 'https://www.ilpost.it/podcasts/timbuctu/ep-323-lanno-record-della-pena-di-morte/', + 'md5': '55d88cc23bcab991639ebcbf1b4c0aa1', + 'info_dict': { + 'id': '3326553', + 'ext': 'mp3', + 'display_id': 'ep-323-lanno-record-della-pena-di-morte', + 'title': 'Ep. 323 – L’anno record della pena di morte', + 'url': 'https://static-prod.cdnilpost.com/wp-content/uploads/2025/05/25/1748196012-timbuctu_250526_v1_-16lufs.mp3', + 'timestamp': 1748235641, + 'upload_date': '20250526', + 'description': 'md5:331514a14779fab06e902160ec8c89ba', + 'duration': 751, + 'availability': 'public', + 'series_id': '233679', + 'thumbnail': 'https://www.ilpost.it/wp-content/uploads/2023/05/19/1684536738-copertina500x500.jpg', }, + }, { 'url': 'https://www.ilpost.it/podcasts/l-invasione/1-avis-akvasas-ka/', 'md5': '43649f002d85e1c2f319bb478d479c40', @@ -47,14 +48,14 @@ class IlPostIE(InfoExtractor): 'series_id': '235598', 'thumbnail': 'https://www.ilpost.it/wp-content/uploads/2023/12/22/1703238848-copertina500x500.jpg', }, - }, - ] + }] def _real_extract(self, url): display_id = self._match_id(url) webpage = self._download_webpage(url, display_id) - episode = self._search_nextjs_data(webpage, display_id)['props']['pageProps']['data']['data']['episode']['data'][0] + episode = self._search_nextjs_data( + webpage, display_id)['props']['pageProps']['data']['data']['episode']['data'][0] return { 'id': str(episode['id']), @@ -74,15 +75,14 @@ def _real_extract(self, url): class IlPostPodcastIE(InfoExtractor): _VALID_URL = r'https?://(?:www\.)?ilpost\.it/podcasts/(?P[^/?#]+)/?(?:[?#]|$)' - _TESTS = [ - { - 'url': 'https://www.ilpost.it/podcasts/basaglia-e-i-suoi/', - 'info_dict': { - 'id': '239295', - 'title': 'Basaglia e i suoi', - }, - 'playlist_mincount': 5, + _TESTS = [{ + 'url': 'https://www.ilpost.it/podcasts/basaglia-e-i-suoi/', + 'info_dict': { + 'id': '239295', + 'title': 'Basaglia e i suoi', }, + 'playlist_mincount': 5, + }, { 'url': 'https://www.ilpost.it/podcasts/morning/', 'info_dict': { @@ -90,27 +90,38 @@ class IlPostPodcastIE(InfoExtractor): 'title': 'Morning', }, 'playlist_mincount': 20, - }, - ] + }] def _real_extract(self, url): display_id = self._match_id(url) + entries = [] + podcast = None - data = self._download_json(f'https://api-prod.ilpost.it/podcast/v1/podcast/{display_id}', display_id, query={'hits': '20'}) - data = self._download_json(f'https://api-prod.ilpost.it/podcast/v1/podcast/{display_id}', display_id, query={'hits': data['head']['data']['total']}) + max_hits = 10000 # found experimentally - podcast = data['data'][0]['parent'] + for page in itertools.count(1): + data = self._download_json( + f'https://api-prod.ilpost.it/podcast/v1/podcast/{display_id}', + display_id, + query={'hits': max_hits, 'pg': page}, + expected_status=500, + ) - entries = [{ - '_type': 'url', - 'ie_key': IlPostIE.ie_key(), - 'url': episode['url'], - **traverse_obj(episode, { - 'episode_id': ('id', {str_or_none}), - 'title': ('title', {clean_html}), - 'description': ('content_html', {clean_html}), - }), - } for episode in traverse_obj(data, ('data', lambda _, v: url_or_none(v['url'])))] + if podcast is None: + podcast = traverse_obj(data, ('data', 0, 'parent')) - return self.playlist_result(entries, - str(podcast['id']), clean_html(podcast.get('title'))) + if data.get('data') is None: + break + + entries += [{ + '_type': 'url', + 'ie_key': IlPostIE.ie_key(), + 'url': episode['url'], + **traverse_obj(episode, { + 'episode_id': ('id', {str_or_none}), + 'title': ('title', {clean_html}), + 'description': ('content_html', {clean_html}), + }), + } for episode in traverse_obj(data, ('data', lambda _, v: url_or_none(v['url'])))] + + return self.playlist_result(entries, str_or_none(podcast.get('id')), clean_html(podcast.get('title')))